[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 #30 from Tomás Cohen Arazi <tomascohen at gmail.com> ---
(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.
> > > 
> > > 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.

+1

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