[Koha-bugs] [Bug 19799] Changing language on OPAC redirects back to homepage

2020-05-03 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19799

David Cook  changed:

   What|Removed |Added

 CC||dc...@prosentient.com.au

--- Comment #21 from David Cook  ---
So interesting seeing the history of Bug 21267!

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 19799] Changing language on OPAC redirects back to homepage

2018-08-30 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19799

Marcel de Rooy  changed:

   What|Removed |Added

   See Also||https://bugs.koha-community
   ||.org/bugzilla3/show_bug.cgi
   ||?id=21299

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 19799] Changing language on OPAC redirects back to homepage

2018-08-30 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19799

Marcel de Rooy  changed:

   What|Removed |Added

 Status|Needs Signoff   |RESOLVED
 Resolution|--- |FIXED

--- Comment #20 from Marcel de Rooy  ---
Fixed by the X-FORWARDED-PROTO approach on bug 21267.
Will resubmit the referer code under a new enhancement.

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 19799] Changing language on OPAC redirects back to homepage

2018-08-30 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19799

Marcel de Rooy  changed:

   What|Removed |Added

  Attachment #77324|0   |1
is obsolete||

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 19799] Changing language on OPAC redirects back to homepage

2018-08-30 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19799

Marcel de Rooy  changed:

   What|Removed |Added

  Attachment #77323|0   |1
is obsolete||

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 19799] Changing language on OPAC redirects back to homepage

2018-08-24 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19799

Tomás Cohen Arazi  changed:

   What|Removed |Added

   See Also||https://bugs.koha-community
   ||.org/bugzilla3/show_bug.cgi
   ||?id=21267

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

[Koha-bugs] [Bug 19799] Changing language on OPAC redirects back to homepage

2018-08-21 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19799

--- Comment #19 from Tomás Cohen Arazi  ---
(In reply to Ulrich Kleiber from comment #18)
> Tomás, I think the mod_rewrite module is responsible for the setting of
> 'SSLOptions +StdEnvVars'. It is required for the RewriteCond Directive.
> 
> https://httpd.apache.org/docs/current/mod/mod_rewrite.html
> ...These variables are available even without setting the StdEnvVars option
> of the SSLOptions directive...
> 
> Using 'RequestHeader set X_FORWARDED_PROTO "https" env=HTTPS' with Koha
> shoudln't be a problem.

Awesome, Uli. I propose we file a new bug for setting what you propose, and we
keep this one for Marcel's enhancement which is still a nice improvement.

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

[Koha-bugs] [Bug 19799] Changing language on OPAC redirects back to homepage

2018-08-21 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19799

--- Comment #18 from Ulrich Kleiber  ---
Tomás, I think the mod_rewrite module is responsible for the setting of
'SSLOptions +StdEnvVars'. It is required for the RewriteCond Directive.

https://httpd.apache.org/docs/current/mod/mod_rewrite.html
...These variables are available even without setting the StdEnvVars option of
the SSLOptions directive...

Using 'RequestHeader set X_FORWARDED_PROTO "https" env=HTTPS' with Koha
shoudln't be a problem.

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

[Koha-bugs] [Bug 19799] Changing language on OPAC redirects back to homepage

2018-08-21 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19799

--- Comment #17 from Ulrich Kleiber  ---
Tomás, we are on debian jessie. 'SSLOptions +StdEnvVars' is only set in
default-ssl.conf, but this site is not enabled. It looks like it is enabled by
default.

/etc$ sudo grep -R StdEnvVars *
apache2/sites-available/default-ssl.conf:   #   o StdEnvVars:
apache2/sites-available/default-ssl.conf:  
SSLOptions +StdEnvVars
apache2/sites-available/default-ssl.conf:  
SSLOptions +StdEnvVars

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

[Koha-bugs] [Bug 19799] Changing language on OPAC redirects back to homepage

2018-08-21 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19799

--- Comment #16 from Tomás Cohen Arazi  ---
(In reply to Ulrich Kleiber from comment #15)
> Tested 'RequestHeader set X_FORWARDED_PROTO "https" env=HTTPS' with http and
> https, everthing seemed to be ok.

Uli, the solution requires SSLOptions +StdEnvVars which I'm not sure is enabled
by default. Did you have to tweak it yourself? What OS/version are you on?

https://httpd.apache.org/docs/2.4/mod/mod_ssl.html

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

[Koha-bugs] [Bug 19799] Changing language on OPAC redirects back to homepage

2018-08-20 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19799

Ulrich Kleiber  changed:

   What|Removed |Added

 CC||ulrich.klei...@bsz-bw.de

--- Comment #15 from Ulrich Kleiber  ---
Tested 'RequestHeader set X_FORWARDED_PROTO "https" env=HTTPS' with http and
https, everthing seemed to be ok.

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 19799] Changing language on OPAC redirects back to homepage

2018-08-20 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19799

--- Comment #14 from Katrin Fischer  ---
We've also tested this line, but only with https:

RequestHeader set X_FORWARDED_PROTO "https" env=HTTPS

Maybe this could be a general solution?

https://httpd.apache.org/docs/2.4/mod/mod_headers.html#requestheader
The RequestHeader directive may be followed by an additional argument, which
may be any of: 
early Specifies early processing. 
env=[!]varname The directive is applied if and only if the environment variable
varname exists.
A ! in front of varname reverses the test, so the directive applies only if
varname is unset.

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 19799] Changing language on OPAC redirects back to homepage

2018-08-20 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19799

--- Comment #13 from Katrin Fischer  ---
We tested the solution Tomas suggested and it works well!

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 19799] Changing language on OPAC redirects back to homepage

2018-08-20 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19799

--- Comment #12 from Marcel de Rooy  ---
This report needs a staff counterpart.

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 19799] Changing language on OPAC redirects back to homepage

2018-08-18 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19799

Tomás Cohen Arazi  changed:

   What|Removed |Added

 CC||tomasco...@gmail.com

--- Comment #11 from Tomás Cohen Arazi  ---
The right way to make things work with our codebase is to uncomment the
following line in both apache-shared-intranet-plack.conf and
apache-shared-opac-plack.conf.

# RequestHeader set X-FORWARDED-PROTO "https"

This header would be used by Plack::Middleware::ReverseProxy to make
$cgi->https work correctly [1]

The problem with that is we didn't manage to automate it for non-tech users.
That's an area we could spend time on!

The approach I aimed for having time to implement, was to split like this:

OPACBaseURL => OPACProto (http/https/gopher) + OPACDomainName + OPACBasePath
(/cgi-bin/koha, preparing the ground for removing the requirement to use that
path). The problem with this is that we of course allow different domains to be
used (a feature).

If we were to add code that workarounds the issue on the codebase, I think we
should better add it as a Plack middleware as we did in
Koha::Middleware::SetEnv.

[1]
https://github.com/plack/Plack/wiki/How-to-detect-reverse-proxy-and-SSL-frontend

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

[Koha-bugs] [Bug 19799] Changing language on OPAC redirects back to homepage

2018-07-30 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19799

--- Comment #10 from Marcel de Rooy  ---
This might help you understand the problem:

>From C4/Auth:
# Set the using_https variable for templates
# FIXME Under Plack the CGI->https method always returns 'OFF'
my $https = $in->{query}->https();
my $using_https = ( defined $https and $https ne 'OFF' ) ? 1 : 0;

If you call CGI->url under Plack, you will get http back which is obviously
wrong if you used https.

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 19799] Changing language on OPAC redirects back to homepage

2018-07-30 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19799

--- Comment #9 from Marcel de Rooy  ---
Caused by bug 18946 on 17.11 and backported

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 19799] Changing language on OPAC redirects back to homepage

2018-07-30 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19799

Marcel de Rooy  changed:

   What|Removed |Added

   Severity|minor   |normal

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 19799] Changing language on OPAC redirects back to homepage

2018-07-30 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19799

--- Comment #7 from Marcel de Rooy  ---
Created attachment 77323
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=77323&action=edit
Bug 19799: Add localReferer function to module, and test

Before implementing the change in opac-changelanguage, we define a new
function (in Koha/Util.pm) and add tests.

Test plan:
Run t/Koha/Util/localReferer.t

Signed-off-by: Marcel de Rooy 

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 19799] Changing language on OPAC redirects back to homepage

2018-07-30 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19799

Marcel de Rooy  changed:

   What|Removed |Added

   Patch complexity|--- |Small patch
 Status|ASSIGNED|Needs Signoff

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 19799] Changing language on OPAC redirects back to homepage

2018-07-30 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19799

--- Comment #8 from Marcel de Rooy  ---
Created attachment 77324
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=77324&action=edit
Bug 19799: Introduce localReferer in opac-changelanguage.pl

Nice and easy replace..

Test plan:
Test changing language from various OPAC locations. (Verify that you are not
redirected to the home page.)
Include language=[some_language] in the URL before changing language.
[Bonus] Test calling opac-changelanguage.pl from another server.

Signed-off-by: Marcel de Rooy 

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 19799] Changing language on OPAC redirects back to homepage

2018-07-30 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19799

Marcel de Rooy  changed:

   What|Removed |Added

   See Also||https://bugs.koha-community
   ||.org/bugzilla3/show_bug.cgi
   ||?id=18946

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 19799] Changing language on OPAC redirects back to homepage

2018-07-30 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19799

--- Comment #6 from Marcel de Rooy  ---
(In reply to Katrin Fischer from comment #4)
> This seems to work alright in my devbox on master. Could someone confirm for
> 18.05?

The bug can be reproduced on 16.11, but on master too.
Depends on factors as using https and Plack.
If you use both Plack and https (which you normally should), you should be able
to reproduce it.

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 19799] Changing language on OPAC redirects back to homepage

2018-07-30 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19799

Marcel de Rooy  changed:

   What|Removed |Added

   Assignee|oleon...@myacpl.org |m.de.r...@rijksmuseum.nl
 Status|NEW |ASSIGNED
 CC||m.de.r...@rijksmuseum.nl
Version|17.11   |master

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 19799] Changing language on OPAC redirects back to homepage

2018-07-25 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19799

Pablo AB  changed:

   What|Removed |Added

 CC||pablo.bian...@gmail.com

--- Comment #5 from Pablo AB  ---
Still happen on 18.05, very annoying.
Not only on search result (opac-search.pl) but anywhere on OPAC, even on user
account (opac-user.pl).

Maybe language should be part of the URL query using ISO 639-1 language code:
?hl=en.

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 19799] Changing language on OPAC redirects back to homepage

2018-06-19 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19799

--- Comment #4 from Katrin Fischer  ---
This seems to work alright in my devbox on master. Could someone confirm for
18.05?

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 19799] Changing language on OPAC redirects back to homepage

2018-05-31 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19799

Katrin Fischer  changed:

   What|Removed |Added

Summary|[16.11] Changing language   |Changing language on OPAC
   |on OPAC redirects back to   |redirects back to homepage
   |homepage|
Version|16.11   |17.11

--- Comment #3 from Katrin Fischer  ---
Also happens in 17.11

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/