[Koha-bugs] [Bug 28317] Remove CGI::Session::Serialize::yaml dependency by using the default serializer

[bugzilla-daemon at bugs.koha-community.org]

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28317

--- Comment #29 from Kyle M Hall <kyle at bywatersolutions.com> ---
(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 now, and I won't have
> > more time to dedicate to this, we are close to the release.
> > We should open a new bug report to reintroduce yaml using YAML::XS, later.
> > Unless someone else has time to do it before the end of the week.
> 
> Surely the least amount of work would be to just keep the YAML::Syck
> dependency for CGI::Session::Serialize::yaml for the time being, and then
> change it to YAML::XS later?
> 
> If we change it to the default serializer now, I think it would much harder
> to switch back to YAML later.

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 internals,
which are subject to change.

-- 
You are receiving this mail because:
You are watching all bug changes.