-
Notifications
You must be signed in to change notification settings - Fork 71
THREESCALE-11725: Update JSON schema validator to V8 #4055
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.
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
THREESCALE-11725: Update JSON schema validator to V8 #4055
Conversation
Where was the change applied?... I can't find in in the apicast repo. |
akostadinov
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This means that we need to run with an updated version of apicast, correct?
This update is nice, just pls merge only after updated version of apicast is in all products we support.
No idea, but @tkan145 confirmed it's live, and I checked and at least |
Not necessarily, just the var
We don't need to update apicast, just the JSON file. However, I don't know what is the value for |
OK, I've found these PRs in But I would like to confirm, @tkan145 , this path is used purely for documentation purposes and the files in it are not in any execution path, right?https://github.com/3scale/APIcast/tree/master/doc/policies_list I understand that the actual policies JSON, when a
I understand that it can be set in the APIManager CR, but is set to |
Not sure you're completely right, but I think you point in the right direction. Maybe this comment can help you: 3scale/APIcast#1530 (comment)
OK, in that case I think we can assume there's no problem for on premises because the next release will include the changes for both porta and apicast and will include a fixed policies JSON. I don't know about RHOAM, it also uses regular releases, right? I'll wait to @akostadinov comments before merging. |
I'm actually not sure this is right:
I don't think we query |
Yes, we don't. |
What this PR does / why we need it:
In order to update
nanoid(#3968) I had to updatereact-jsonschema-form@1.8.1to@rjsf/core@5.24.1. This package is used from the Policies screen. We have the policies defined in JSON schema format and this package transforms then into HTML forms. The package includes a validator, which last version is V8, but I had to use the deprecated V6 version because ourpolicies.jsonwas being considered inavalid by V8, check this comment where I explain the issue.After suggesting some changes to our
policies.json, a new version has been released which solves the problem, so we are now free to update to the V8 validator.Which issue(s) this PR fixes
https://issues.redhat.com/browse/THREESCALE-11725
Verification steps
The policies screen should work normally after the change. You can add and remove policies with the developer tools opened, to verify no errors appears in the console.
In particular, the URL Rewriting policy was impossible to add using the new validator, now you should be able to add it and load it without problems.