- Koha-devel - lists.koha-community.org

Re: [Koha] Re: MARC support
by Joshua Brindle 15 Jan '03

15 Jan '03

You are right, but consider this practically every cgi page includes C4::Auth. Now, say i want to put in ldap authentication, i have to go modify Auth.pm to do what I want, and if the API changes i have to go fix my own auth.pm to continue working. the correct way to do this is to have a wrapper that does the interface between koha and the authentication module. you should never have to modify modules that come with koha to make a new authentication method, it should work like this app includes C4::Auth initialize autnetication wrapper with optional argument (being the authentication module to use) either C4::Auth uses the module in the argument, or reverts to the default DBI authentication the default DBI authentication in C4 would inherit the methods in the optional authentication module so that the authentication module can do as much or as little as needed. For example, ldap would only verify the password, and get the card number, but the other fuctions like userflags, permissions, etc would not be overwritten and thus still handled by the DBI authentication. Here is another example.. Most universities have the patron information in another (possibly proprietary) database. The proprietary database might would have the patrons card number, and name information, but nothing else. so this users custom authentication module would only overwrite the getborrowernumber method, but no other. that user should be able to drop in that 1 method and allow the default DBI authentication to do the rest. Now, c4::auth does contain all the authentication but if the user wants to change authentication methods he'll have to overwrite auth.pm and possibly repeat a lot of it, or spend considable time changing all the pages to include something other than C4::Auth. I may be rambling now, but this is how it should work, so that the actual program is never changed so that api changes don't destroy the systems. This should work on things other than authentication too, such as data storage. Thanks, and i hope i didn't ramble this into confusion :) Joshua Brindle UNIX Administrator Southern Nazarene University >>> paul POULAIN <paul.poulain(a)free.fr> 01/15/03 09:44AM >>> Joshua Brindle a écrit: What is the timeline for the authentication stuff? I've gone through some of the authentication stuff and it seems deeply embedded (ie: not modular) to do this right the authentication should be completely abstracted from the app and allow a simple drop in authentication replacement. If this were the case i'd happilly write the ldap stuff. Are there plans to modularize all this stuff? Thanks.... Unless I'm missing something, everything around auth is done in C4::Auth.pm package, so i don't understand why it's not modular ??? -- Paul POULAINConsultant indépendant en logiciels libresresponsable francophone de koha (SIGB libre http://www.koha-fr.org)

1 0

Re: [Koha] Re: MARC support
by paul POULAIN 15 Jan '03

15 Jan '03

Joshua Brindle a écrit: > You are right, but consider this <snip> > I may be rambling now, but this is how it should work, so that the > actual program is never changed so that api changes don't destroy the > systems. This should work on things other than authentication too, > such as data storage. Thanks, and i hope i didn't ramble this into > confusion :) Too late, you did... I'm confused now :-) I still don't understand what you mean : if you change the auth process, you just have to change get_template_and_user function, which does everything. Anyway, maybe i'm a poor perl developper :-) -- Paul POULAIN Consultant indépendant en logiciels libres responsable francophone de koha (SIGB libre http://www.koha-fr.org)

1 0

buildrelease script sits forever
by Benedict 15 Jan '03

15 Jan '03

Hi, The buildrelease script goes like this: Updating your checked-out copy of the 'koha' CVS files. You may need to enter your SourceForge password. Using ../koha. cvs server: Updating . cvs server: Updating info cvs server: Updating lang-datas cvs server: Updating lang-datas/fr cvs server: Updating marc_datas cvs server: Updating marc_datas/marc21_en cvs server: Updating marc_datas/unimarc_fr Updating your checked-out copy of the 'koha-html' CVS files. You may need to enter your SourceForge password. Using ../../koha-html. cvs server: Updating . cvs server: Updating intranet-html cvs server: Updating intranet-html/acquisitions cvs server: Updating intranet-html/admin cvs server: Updating intranet-html/catalogue cvs server: Updating intranet-html/images cvs server: Updating intranet-html/images/numbers cvs server: Updating intranet-html/issues cvs server: Updating intranet-html/members cvs server: Updating intranet-html/opac cvs server: Updating intranet-html/opac/images cvs server: Updating intranet-html/reports cvs server: Updating opac-html cvs server: Updating opac-html/cat cvs server: Updating opac-html/images cvs server: Updating opac-html/includes cvs server: Updating opac-html/members And sits like that forever. any clue why? Benedict Is there a command switch to skip the update? I can do it manually from cervisia to make sure I've got the latest copy.

2 1

Re: [Koha] Re: MARC support
by paul POULAIN 15 Jan '03

15 Jan '03

Joshua Brindle a écrit: > What is the timeline for the authentication stuff? > I've gone through some of the authentication stuff and it seems deeply > embedded (ie: not modular) > to do this right the authentication should be completely abstracted > from the app and allow a simple drop in authentication replacement. If > this were the case i'd happilly write the ldap stuff. Are there plans > to modularize all this stuff? Thanks.... Unless I'm missing something, everything around auth is done in C4::Auth.pm package, so i don't understand why it's not modular ??? -- Paul POULAIN Consultant indépendant en logiciels libres responsable francophone de koha (SIGB libre http://www.koha-fr.org)

1 0

Regressions in the latest CVS
by acli＠ada.dhs.org via news-to-mail gateway 15 Jan '03

15 Jan '03

Hi, there are regressions in the CVS (bugs 129 and 146 re-appeared). I tried to make my old patch work in the current CVS version (leaving out some things that other people corrected in some other way). Please check the patch & commit to CVS if ok. Some of the fixes are quoting related; quoting bugs are usually considered security holes, though quoting bugs in an installer are just nuisances and won't affect everyone. In the patch there is a change in the way /etc is referenced; I did this because if this is not done the $::etcdir variable is basically pointless. Perhaps my patches are too big, but it is rather difficult to make small patches when I need to make the installer work before I test anything. I can't know if my patch works until I make it all work. ------------ cut here --------- 8< -------------------------- diff -u koha-1.3.2.orig/Install.pm koha-1.3.2/Install.pm --- koha-1.3.2.orig/Install.pm Thu Jan 9 12:40:22 2003 +++ koha-1.3.2/Install.pm Sun Jan 12 23:28:16 2003 @@ -28,6 +28,7 @@ @ISA = qw(Exporter); @EXPORT = qw( &checkperlmodules + &checkabortedinstall &getmessage &showmessage &releasecandidatewarning @@ -134,7 +135,7 @@ = Koha already installed = ========================== -It looks like Koha is already installed on your system (/etc/koha.conf exists +It looks like Koha is already installed on your system (%s/koha.conf exists already). If you would like to upgrade your system to %s, please use the koha.upgrade script in this directory. @@ -178,7 +179,7 @@ This release of Koha has a new authentication module. If you are not already using basic authentication on your intranet, you will be required to log in to access some of the features of the intranet. You can log in using the userid -and password from the /etc/koha.conf configuration file at any time. Use the +and password from the %s/koha.conf configuration file at any time. Use the "Members" module to add passwords for other accounts and set their permissions. Press the <ENTER> key to continue: |; @@ -219,6 +220,34 @@ # +# Assuming that Koha will be installed on a modern Unix with symlinks, +# it is possible to code the installer so that aborted installs can be +# detected. In case of such an event we can do our best to "roll back" +# the aborted install. +# +# FIXME: The "roll back" is not complete! +# + +sub checkabortedinstall { + if (-l("$::etcdir/koha.conf") + && readlink("$::etcdir/koha.conf") =~ /\.tmp$/ + ) { + print qq| +I have detected that you tried to install Koha before, but the installation +was aborted. I will try to continue, but there might be problems if the +database is already created. + +|; + print "Please press <ENTER> to continue: "; + <STDIN>; + + # Remove the symlink after the <STDIN>, so the user can back out + unlink "$::etcdir/koha.conf" + || die "Failed to remove incomplete $::etcdir/koha.conf: $!\n"; + } +} + +# # Test for Perl and Modules # # @@ -507,6 +536,8 @@ Please provide the name of the user, who will have full administrative rights to the %s database, when authenticating from %s. +This user will also be used to access Koha's INTRANET interface. + Database user [%s]: |; $messages->{'DatabasePassword'}->{en}=qq| @@ -602,7 +633,7 @@ I was not able to determine the user that Apache is running as. This information is necessary in order to set the access privileges correctly on -/etc/koha.conf. This user should be set in one of the Apache configuration +%s/koha.conf. This user should be set in one of the Apache configuration files using the "User" directive. Enter the Apache userid: |; @@ -637,7 +668,7 @@ if ($#confpossibilities==-1) { my $message=getmessage('NoApacheConfFiles'); my $choice=''; - until (-f $choice) { + until (-f $::realhttpdconf) { $choice=showmessage($message, "free", 1); if (-f $choice) { $::realhttpdconf=$choice; @@ -660,7 +691,7 @@ } else { $::realhttpdconf=$confpossibilities[0]; } - unless (open (HTTPDCONF, $::realhttpdconf)) { + unless (open (HTTPDCONF, "<$::realhttpdconf")) { warn "Insufficient privileges to open $::realhttpdconf for reading.\n"; sleep 4; } @@ -676,7 +707,7 @@ unless ($::httpduser) { - my $message=getmessage('EnterApacheUser'); + my $message=getmessage('EnterApacheUser', [$::etcdir]); until (length($::httpduser) && getpwnam($::httpduser)) { $::httpduser=showmessage($message, "free", ''); if (length($::httpduser)>0) { @@ -807,11 +838,12 @@ Press <ENTER> to continue: |; sub updateapacheconf { - my $logfiledir=`grep ^ErrorLog $::realhttpdconf`; + my $logfiledir=`grep ^ErrorLog "$::realhttpdconf"`; chomp $logfiledir; if ($logfiledir) { - $logfiledir=~m#ErrorLog (.*)/[^/]*$#; + $logfiledir=~m#ErrorLog (.*)/[^/]*$# + or die "Can't parse ErrorLog directive\n"; $logfiledir=$1; } @@ -824,7 +856,7 @@ my $httpdconf; my $envmodule=0; my $includesmodule=0; - open HC, $::realhttpdconf; + open HC, "<$::realhttpdconf"; while (<HC>) { if (/^\s*#\s*LoadModule env_module /) { s/^\s*#\s*//; @@ -856,7 +888,7 @@ - if (`grep 'VirtualHost $::servername' $::realhttpdconf`) { + if (`grep 'VirtualHost $::servername' "$::realhttpdconf"`) { showmessage(getmessage('ApacheAlreadyConfigured', [$::realhttpdconf, $::realhttpdconf]), 'PressEnter'); return; } else { @@ -933,6 +965,10 @@ I can set it up so that the Intranet/Librarian site is password protected using Apache's Basic Authorization. +This is going to be phased out very soon. However, setting this up can provide +an extra layer of security before the new authentication system is completely +in place. + Would you like to do this ([Y]/N): |; $messages->{'BasicAuthUsername'}->{en}="Please enter a userid for intranet access [%s]: "; @@ -954,7 +990,7 @@ ($apacheauthpassword) = showmessage(getmessage('BasicAuthPasswordWasBlank'), 'none', '', 1); } } - open AUTH, ">/etc/kohaintranet.pass"; + open AUTH, ">$::etcdir/kohaintranet.pass"; my $chars='ABCDEFGHIJKLMNOPQRSTUVWXYZabcdefghijklmnopqrstuvwxyz0123456789'; my $salt=substr($chars, int(rand(length($chars))),1); $salt.=substr($chars, int(rand(length($chars))),1); @@ -964,7 +1000,7 @@ print SITE <<EOP <Directory $::intranetdir> - AuthUserFile /etc/kohaintranet.pass + AuthUserFile $::etcdir/kohaintranet.pass AuthType Basic AuthName "Koha Intranet (for librarians only)" Require valid-user @@ -1018,7 +1054,7 @@ hostname=$::hostname user=$::user pass=$::pass -includes=$::intranetdir/htdocs/includes +includes=$::opacdir/htdocs/includes intranetdir=$::intranetdir opacdir=$::opacdir kohalogdir=$::kohalogdir @@ -1026,6 +1062,8 @@ httpduser=$::httpduser intrahtdocs=$::intranetdir/htdocs/intranet-tmpl opachtdocs=$::opacdir/htdocs/opac-tmpl +#XXX I had: intrahtdocs=$::intranetdir/cgi-bin/koha-tmpl/intranet-tmpl +#XXX I had: opachtdocs=$::intranetdir/cgi-bin/koha-tmpl/opac-tmpl |; close(SITES); umask($old_umask); @@ -1049,7 +1087,7 @@ To allow us to create the koha database please supply your mysql server's root user password: -Enter MySql root user password: |; +Enter MySQL root user password: |; $messages->{'InvalidMysqlRootPassword'}->{en}="Invalid Password. Please try again."; @@ -1058,7 +1096,7 @@ = CREATING DATABASE = ===================== -Creating the MySql database for Koha... +Creating the MySQL database for Koha... |; @@ -1227,6 +1265,9 @@ |; sub updatedatabase { + # At this point, $::etcdir/koha.conf must exist, for C4::Context + # We must somehow temporarily enable $::etcdir/koha.conf. A symlink can + # do this & at the same time facilitate detection of aborted installs. my $result=system ("perl -I $::intranetdir/modules scripts/updater/updatedatabase"); if ($result) { print "Problem updating database...\n"; @@ -1236,11 +1277,11 @@ my $response=showmessage(getmessage('UpdateMarcTables'), 'restrictchar 123', '1'); if ($response == 1) { - system("cat script/misc/marc_datas/marc21_en/structure_def.sql | $::mysqldir/bin/mysql -u$::mysqluser -p$::mysqlpass $::dbname"); + system("cat scripts/misc/marc_datas/marc21_en/structure_def.sql | $::mysqldir/bin/mysql -u$::mysqluser $::mysqlpass_quoted $::dbname"); } if ($response == 2) { - system("cat scripts/misc/marc_datas/unimarc_fr/structure_def.sql | $::mysqldir/bin/mysql -u$::mysqluser -p$::mysqlpass $::dbname"); - system("cat scripts/misc/lang-datas/fr/stopwords.sql | $::mysqldir/bin/mysql -u$::mysqluser -p$::mysqlpass $::dbname"); + system("cat scripts/misc/marc_datas/unimarc_fr/structure_def.sql | $::mysqldir/bin/mysql -u$::mysqluser $::mysqlpass_quoted $::dbname"); + system("cat scripts/misc/lang-datas/fr/stopwords.sql | $::mysqldir/bin/mysql -u$::mysqluser $::mysqlpass_quoted $::dbname"); } my $result=system ("perl -I $::intranetdir/modules scripts/marc/updatedb2marc.pl"); @@ -1249,16 +1290,14 @@ exit; } - print "\n\nFinished updating database. Press <ENTER> to continue..."; + print "\n\nFinished basic updating of database. Press <ENTER> to continue..."; <STDIN>; } sub populatedatabase { my $response=showmessage(getmessage('SampleData'), 'yn', 'n'); if ($response =~/^y/i) { - system("gunzip sampledata-1.2.gz"); - system("cat sampledata-1.2 | $::mysqldir/bin/mysql -u$::mysqluser $::mysqlpass_quoted $::dbname"); - system("gzip -9 sampledata-1.2"); + system("gunzip -d < sampledata-1.2.gz | $::mysqldir/bin/mysql -u$::mysqluser $::mysqlpass_quoted $::dbname"); system("$::mysqldir/bin/mysql -u$::mysqluser $::mysqlpass_quoted $::dbname -e \"insert into branches (branchcode,branchname,issuing) values ('MAIN', 'Main Library', 1)\""); system("$::mysqldir/bin/mysql -u$::mysqluser $::mysqlpass_quoted $::dbname -e \"insert into branchrelations (branchcode,categorycode) values ('MAIN', 'IS')\""); system("$::mysqldir/bin/mysql -u$::mysqluser $::mysqlpass_quoted $::dbname -e \"insert into branchrelations (branchcode,categorycode) values ('MAIN', 'CU')\""); @@ -1333,7 +1372,7 @@ sub loadconfigfile { my %configfile; - open (KC, "/etc/koha.conf"); + open (KC, "<$::etcdir/koha.conf"); while (<KC>) { chomp; (next) if (/^\s*#/); diff -u koha-1.3.2.orig/installer.pl koha-1.3.2/installer.pl --- koha-1.3.2.orig/installer.pl Sun Jan 5 00:40:29 2003 +++ koha-1.3.2/installer.pl Sun Jan 12 22:17:48 2003 @@ -35,7 +35,7 @@ releasecandidatewarning(); } -#checkabortedinstall(); +checkabortedinstall(); if (-e "$::etcdir/koha.conf") { $::installedversion=`grep kohaversion= $::etcdir/koha.conf`; ------------ cut here --------- 8< -------------------------- Best regards, -- Ambrose Li <a.c.li(a)ieee.org> http://ada.dhs.org/~acli/cmcc/ http://www.cccgt.org/ DRM is theft - We are the stakeholders

3 2

[Bug 158] FreeBSD 5.0 depricates chown [user].[group] breaking installer.pm
by bugzilla-daemon＠wilbur.katipo.co.nz 15 Jan '03

15 Jan '03

http://bugs.koha.org/cgi-bin/bugzilla/show_bug.cgi?id=158 paul.poulain(a)free.fr changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |RESOLVED Resolution| |FIXED ------- You are receiving this mail because: ------- You are the QA contact for the bug, or are watching the QA contact.

1 0

[Bug 157] Installing sample data gives error
by bugzilla-daemon＠wilbur.katipo.co.nz 15 Jan '03

15 Jan '03

http://bugs.koha.org/cgi-bin/bugzilla/show_bug.cgi?id=157 paul.poulain(a)free.fr changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |RESOLVED Resolution| |FIXED ------- Additional Comments From paul.poulain(a)free.fr 2003-01-16 02:02 ------- fixed : was due to a 6th column in itemtypes table ------- You are receiving this mail because: ------- You are the QA contact for the bug, or are watching the QA contact.

1 0

installation very difficult
by Benedict 15 Jan '03

15 Jan '03

Hi, I am getting a lot of different errors after installing Koha. [Tue Jan 14 22:13:31 2003] [error] [client 127.0.0.1] Premature end of script headers: /usr/local/koha/intranet/cgi-bin/acqui.simple/marcimport.pl Tag " " is not a valid tag. at /usr/local/koha/intranet/modules/C4/Biblio.pm line 590 [Tue Jan 14 22:17:40 2003] [error] [client 127.0.0.1] Premature end of script headers: /usr/local/koha/intranet/cgi-bin/MARCdetail.pl [root@localhost intranet-tmpl]# /etc/koha.conf database=Koha hostname=localhost user=kohaadmin pass=xxxxxxx includes=/usr/local/koha/intranet/htdocs/intranet-tmpl/default/pl/includes <<-- ! Is this eight for me? intranetdir=/usr/local/koha/intranet opacdir=/usr/local/koha/opac kohalogdir=/var/log/koha kohaversion=RC1 httpduser=apache intrahtdocs=/usr/local/koha/intranet/htdocs/intranet-tmpl opachtdocs=/usr/local/koha/opac/htdocs/opac-tmpl I modified the includes= directive but I am not sure this is correct. Are intrahtdocs and opachtdocs OK? Are perl scripts aware that they should use ./defaults/pl directory for polish translation ??? Thanks for any answers. Benedict

2 1

another error
by Benedict 14 Jan '03

14 Jan '03

Is this my misconfiguration? ===================== Undefined subroutine CGI::dump at /usr/local/koha/intranet/cgi-bin/placerequest.pl line 76 Global symbol "$flaginfotextEOF" requires explicit package name at /usr/local/koha/intranet/cgi-bin/circ/returns.pl line 368. Global symbol "$returneditemstable" requires explicit package name at /usr/local/koha/intranet/cgi-bin/circ/returns.pl line 368. Global symbol "$returneditemstable" requires explicit package name at /usr/local/koha/intranet/cgi-bin/circ/returns.pl line 409. Global symbol "$returneditemstable" requires explicit package name at /usr/local/koha/intranet/cgi-bin/circ/returns.pl line 424. Global symbol "$returneditemstable" requires explicit package name at /usr/local/koha/intranet/cgi-bin/circ/returns.pl line 474. Missing right curly or square bracket at /usr/local/koha/intranet/cgi-bin/circ/returns.pl line 491, at end of line syntax error at /usr/local/koha/intranet/cgi-bin/circ/returns.pl line 491, at EOF Execution of /usr/local/koha/intranet/cgi-bin/circ/returns.pl aborted due to compilation errors. [Tue Jan 14 22:41:42 2003] [error] [client 127.0.0.1] Premature end of script headers: /usr/local/koha/intranet/cgi-bin/circ/returns.pl =========================== When trying to add a new memeber I get this: ===================== syntax error at /usr/local/koha/intranet/cgi-bin/newmember.pl line 162, at EOF Execution of /usr/local/koha/intranet/cgi-bin/newmember.pl aborted due to compilation errors. [Tue Jan 14 22:45:25 2003] [error] [client 127.0.0.1] Premature end of script headers: /usr/local/koha/intranet/cgi-bin/newmember.pl ====================== Benedict

4 3

[Bug 158] New: FreeBSD 5.0 depricates chown [user].[group] breaking installer.pm
by bugzilla-daemon＠wilbur.katipo.co.nz 14 Jan '03

14 Jan '03

http://bugs.koha.org/cgi-bin/bugzilla/show_bug.cgi?id=158 Summary: FreeBSD 5.0 depricates chown [user].[group] breaking installer.pm Product: Koha Version: CVS Platform: All OS/Version: FreeBSD Status: NEW Severity: normal Priority: P2 Component: Installation AssignedTo: tonnesen(a)cmsd.bc.ca ReportedBy: micheas(a)micheas.dyns.net QAContact: koha-devel(a)lists.sourceforge.net In order to support '.' in user names FreeBSD has dropped Support for '.' as a seperator in chown. Only the ':' colon must be used. (chown [user]:[group] [file]) This may effect Mac os/X as well. (It almost certainly will in the futur.) This affects the folloeing files an lines Install.pm: system("chown -R root.$::httpduser $::opacdir"); Install.pm: system("chown -R root.$::httpduser $::intranetdir"); koha.upgrade:system("chown -R root.$httpduser $::opacdir"); koha.upgrade:system("chown -R root.$httpduser $::intranetdir"); which I belive need changed to: Install.pm: system("chown -R root:$::httpduser $::opacdir"); Install.pm: system("chown -R root:$::httpduser $::intranetdir"); koha.upgrade:system("chown -R root:$httpduser $::opacdir"); koha.upgrade:system("chown -R root:$httpduser $::intranetdir"); Thamks, Micheas ------- You are receiving this mail because: ------- You are the QA contact for the bug, or are watching the QA contact.

1 0