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
fix: replace 'blocklist' and 'whitelist' with 'blocked' and 'allowed' #1077
Conversation
I admit I did the changes almost “blindfolded”. I didn't run any tests or otherwise verify that it works. I hope someone can tell me how to do that, or do it for me 😄 |
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.
A bunch of comments.
Validation is failing, I'll try it locally
vaadin-spring/src/main/java/com/vaadin/flow/spring/VaadinConfigurationProperties.java
Show resolved
Hide resolved
vaadin-spring/src/main/java/com/vaadin/flow/spring/VaadinConfigurationProperties.java
Show resolved
Hide resolved
vaadin-spring/src/main/java/com/vaadin/flow/spring/VaadinConfigurationProperties.java
Show resolved
Hide resolved
vaadin-spring/src/main/java/com/vaadin/flow/spring/VaadinConfigurationProperties.java
Show resolved
Hide resolved
vaadin-spring/src/main/java/com/vaadin/flow/spring/VaadinServletContextInitializer.java
Outdated
Show resolved
Hide resolved
The same changes should be applied also on Flow repository. |
vaadin-spring/src/main/java/com/vaadin/flow/spring/VaadinConfigurationProperties.java
Outdated
Show resolved
Hide resolved
vaadin-spring/src/main/java/com/vaadin/flow/spring/VaadinConfigurationProperties.java
Outdated
Show resolved
Hide resolved
vaadin-spring/src/main/java/com/vaadin/flow/spring/VaadinConfigurationProperties.java
Outdated
Show resolved
Hide resolved
vaadin-spring/src/main/java/com/vaadin/flow/spring/VaadinConfigurationProperties.java
Outdated
Show resolved
Hide resolved
vaadin-spring/src/main/java/com/vaadin/flow/spring/VaadinServletContextInitializer.java
Outdated
Show resolved
Hide resolved
Co-authored-by: Marco Collovati <marco@vaadin.com>
Co-authored-by: Marco Collovati <marco@vaadin.com>
Quality Gate failedFailed conditions C Reliability Rating on New Code (required ≥ A) See analysis details on SonarCloud Catch issues before they fail your Quality Gate with our IDE extension SonarLint |
Shouldn't this be done in https://github.com/vaadin/flow/tree/main/vaadin-spring ? This repository is for old Flow versions |
I'll create a PR in the flow repo |
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.
The PR looks good, but tests are still failing, although they pass locally.
I'm looking into it, but in the meanwhile I'll create a specular PR on the Flow repository
Tests failed because of issues with pnpm cache; browser console reported messages like
Cleaning pnpm cache before executing the validation fixed the job |
@mshabarov LGTM, I don't know if you also want to take a look at it |
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.
Looks good to me.
"Blocked-packages" and "allowed-packages" names are inline with what we previously discussed in Slack.
We have to pick this into 19.1 branch as well. |
Hi @jouni , this commit cannot be picked to 19.1 by this bot, can you take a look and pick it manually? |
…#1077) --------- Co-authored-by: Marco Collovati <marco@vaadin.com>
Thanks for helping out with the finalization tasks! |
Fixes #645
Mark the existing methods as deprecated and introduce methods with new names.
Fall back to the old configuration properties if the new ones aren't used.