Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Tried to load unspecified class: Poll (Psych::DisallowedClass): #145

Open
JoJoDeveloping opened this issue Nov 2, 2022 · 1 comment · May be fixed by #152
Open

Tried to load unspecified class: Poll (Psych::DisallowedClass): #145

JoJoDeveloping opened this issue Nov 2, 2022 · 1 comment · May be fixed by #152

Comments

@JoJoDeveloping
Copy link

Updating ruby apparently breaks things. See more here: ruby/psych#503

There are two possible solutions. One is to change YAML::load to YAML::unsafe_load. The better option is to add this to the ruby config:

config.active_record.yaml_column_permitted_classes = [Poll, /* maybe more */]

Unfortunately, I have no idea where the "Ruby config" would even be located for dudle. Hence I can't really offer a fix.
@dl8dtl
Copy link

dl8dtl commented May 21, 2023

I replaced all (three) occurences of YAML::load by YAML::unsafe_load and can confirm this is suitable as a workaround. Alas, I am completely illiterate in Ruby, so no idea about that config file as well.

@xtaran xtaran mentioned this issue Oct 9, 2023
Open
xtaran added a commit to xtaran/dudle that referenced this issue Oct 9, 2023
@xtaran
Compat with newer Psych: Use safe_load with permitted_classes Poll an…
f761ae3 
…d TimePollHead

Fixes kellerben#145, kellerben#151
@xtaran xtaran linked a pull request Oct 9, 2023 that will close this issue
Open
xtaran added a commit to xtaran/dudle that referenced this issue Oct 10, 2023
@xtaran
Compat with newer Psych: Use safe_load with permitted_classes Poll an…
52e95bc 
…d TimePollHead

Fixes kellerben#145, fixes kellerben#151

Based on the discussion in kellerben#145, @dl8dtl's initial patch suggestion
and reading https://docs.ruby-lang.org/en/master/Psych.html to
understand the remaining open question by @JoJoDeveloping where the
config needs to go.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Compat with newer Psych: Use safe_load with permitted_classes Poll and TimePollHead xtaran/dudle
2 participants
@JoJoDeveloping @dl8dtl