https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28317
--- Comment #43 from Joonas Kylmälä ---
(In reply to Joonas Kylmälä from comment #41)
> On Koha-testing-docker I get randomly the following error when trying to
> select a new branch on intranet (/cgi-bin/koha/circ/set-library.pl):
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28317
Joonas Kylmälä changed:
What|Removed |Added
Blocks||28489
Referenced Bugs:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28317
--- Comment #42 from Joonas Kylmälä ---
(In reply to Joonas Kylmälä from comment #41)
> On Koha-testing-docker I get randomly the following error when trying to
> select a new branch on intranet (/cgi-bin/koha/circ/set-library.pl):
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28317
--- Comment #41 from Joonas Kylmälä ---
On Koha-testing-docker I get randomly the following error when trying to select
a new branch on intranet (/cgi-bin/koha/circ/set-library.pl):
CGI::Session::Driver::mysql::store(): DBI
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28317
Fridolin Somers changed:
What|Removed |Added
CC|
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28317
Jonathan Druart changed:
What|Removed |Added
Keywords|rel_21_05_candidate |
--
You are receiving
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28317
--- Comment #39 from Jonathan Druart ---
Pushed to master for 21.05, thanks to everybody involved!
--
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28317
Jonathan Druart changed:
What|Removed |Added
Status|Passed QA |Pushed to master
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28317
Jonathan Druart changed:
What|Removed |Added
Status|In Discussion |Passed QA
--
You are
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28317
--- Comment #38 from David Cook ---
If we're set on removing the YAML and YAML::Syck dependencies, then I say we
move this back to "Passed QA". While in theory we could use YAML::XS with
CGI::Session::Serialize::yaml, the Debian
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28317
--- Comment #37 from David Cook ---
Interestingly, CGI::Session::Serialize::default writes out a Perl data
structure and then uses the equivalent of eval() to inject it back into Perl:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28317
--- Comment #36 from David Cook ---
Interestingly, CGI::Session::Serialize::default writes out a Perl data
structure and then uses the equivalent of eval() to inject it back into Perl:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28317
--- Comment #35 from David Cook ---
Well, I just looked at CGI::Session::Serialize::default and did a little test
using its freeze method. The data ends up looking like this:
$D = {'firstname' => 'David','surname' => 'Cook','id' =>
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28317
--- Comment #34 from David Cook ---
(In reply to David Cook from comment #33)
> (In reply to Julian Maurice from comment #24)
> > If we really want to stick with YAML, I'd prefer Koha to have its own
> > implementation rather than
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28317
--- Comment #33 from David Cook ---
(In reply to Julian Maurice from comment #24)
> If we really want to stick with YAML, I'd prefer Koha to have its own
> implementation rather than relying on an undocumented "feature" (from my
>
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28317
--- Comment #32 from David Cook ---
(In reply to Kyle M Hall from comment #29)
> Wouldn't it make more sense for your to add an API endpoint for outputting
> deserialized session info? That way you aren't messing with Koha's
>
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28317
--- Comment #31 from Victor Grousset/tuxayo ---
(In reply to David Cook from comment #26)
> My third-party apps neither use Perl nor have the capacity to use Perl, so
> that's not an option :p.
Would writing a small Perl proxy(is
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28317
--- Comment #30 from Tomás Cohen Arazi ---
(In reply to Kyle M Hall from comment #29)
> (In reply to David Cook from comment #27)
> > (In reply to Jonathan Druart from comment #25)
> > > David, thanks for the investigation.
> > >
>
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28317
--- Comment #29 from Kyle M Hall ---
(In reply to David Cook from comment #27)
> (In reply to Jonathan Druart from comment #25)
> > David, thanks for the investigation.
> >
> > All what you suggest is more work than what we have
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28317
Jonathan Druart changed:
What|Removed |Added
Comment #28 is|1 |0
private|
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28317
--- Comment #27 from David Cook ---
(In reply to Jonathan Druart from comment #25)
> David, thanks for the investigation.
>
> All what you suggest is more work than what we have now, and I won't have
> more time to dedicate to
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28317
--- Comment #26 from David Cook ---
(In reply to Julian Maurice from comment #24)
> (In reply to David Cook from comment #20)
> > I have third-party apps that can use Koha session data by parsing the YAML.
> > It looks like I'll
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28317
--- Comment #25 from Jonathan Druart ---
David, thanks for the investigation.
All what you suggest is more work than what we have now, and I won't have more
time to dedicate to this, we are close to the release.
We should open a
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28317
--- Comment #24 from Julian Maurice ---
(In reply to David Cook from comment #20)
> I have third-party apps that can use Koha session data by parsing the YAML.
> It looks like I'll have to think of something much more complicated to
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28317
David Cook changed:
What|Removed |Added
Status|Passed QA |In Discussion
--
You are
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28317
--- Comment #23 from David Cook ---
I've opened a couple issues at
https://rt.cpan.org/Dist/Display.html?Name=CGI-Session-Serialize-yaml regarding
documentation and module options.
--
You are receiving this mail because:
You are
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28317
--- Comment #22 from David Cook ---
Interestingly, Tina Muller who has taken over CGI::Session::Serialize::yaml is
also the maintainer of YAML::XS.
Ahhh CGI::Session::Serialize::yaml also has an undocumented feature. If you
look
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28317
--- Comment #21 from David Cook ---
(In reply to Julian Maurice from comment #0)
> Recently we replaced every use of YAML::Syck or YAML by YAML::XS, and
> removed YAML::Syck and YAML from cpanfile.
> But we use
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28317
--- Comment #20 from David Cook ---
(In reply to Jonathan Druart from comment #3)
> Why don't we use CGI::Session::Serialize::default instead?
It's a shame that we're losing the YAML serialization.
As noted on
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28317
Julian Maurice changed:
What|Removed |Added
Status|Signed Off |Passed QA
--- Comment #19
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28317
Julian Maurice changed:
What|Removed |Added
Attachment #120839|0 |1
is obsolete|
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28317
Kyle M Hall changed:
What|Removed |Added
Attachment #120837|0 |1
is obsolete|
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28317
Kyle M Hall changed:
What|Removed |Added
Status|Needs Signoff |Signed Off
--
You are
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28317
Jonathan Druart changed:
What|Removed |Added
Attachment #120836|0 |1
is obsolete|
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28317
Jonathan Druart changed:
What|Removed |Added
Summary|CGI::Session::Serialize::ya |Remove
35 matches
Mail list logo