search

rails / activerecord-session_store

Active Record's Session Store extracted from Rails
MIT License
541 stars 188 forks source link

Remove multi_json as dependency in favor of std-lib json #210

Open robsdudeson opened 8 months ago

robsdudeson commented 8 months ago

multi_json is End of Life for over a decade now

we should use the ruby std-lib json instead

otherguy commented 4 weeks ago

Also implemented here: https://github.com/rails/activerecord-session_store/pull/213

robsdudeson commented 4 weeks ago

I've moved on from a project where this was a problem. I'm happy to go with your approach, though I'm not a maintainer on this.

otherguy commented 4 weeks ago

Yeah, our PRs are identical. I just wanted to cross link it here, in case someone else stumbles upon this issue.

Perhaps @byroot could take a look at merging one of them?