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.

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

[FW][FIX] payment: prevent ACL denial when unlinking onboarding wizard #45363

Conversation

@fw-bot
Copy link
Contributor

fw-bot commented Feb 14, 2020

Since wizards shall not be unlinked by anyone, but as we want to get rid of this specific one immediately to get rid of eventual credentials, do it with elevated privileges.

Before this commit, the behavior was an authorization error when trying to applying the payment method, in a fresh invoicing configuration.

Forward-Port-Of: #45327

Since wizards shall not be unlinked by anyone, but as we want to get rid
of this specific one immediately to get rid of eventual credentials,
do it with elevated privileges.

Before this commit, the behavior was an authorization error when trying
to applying the payment method, in a fresh invoicing configuration.

X-original-commit: c8ee9c7
@fw-bot

This comment has been minimized.

Copy link
Contributor Author

fw-bot commented Feb 14, 2020

Ping @madprog, @mart-e
This PR targets master and is the last of the forward-port chain.

To merge the full chain, say

@fw-bot r+

More info at https://github.com/odoo/odoo/wiki/Mergebot-and-Forwardbot#forward-port

@mart-e

This comment has been minimized.

Copy link
Contributor

mart-e commented Feb 14, 2020

@fw-bot r+

@robodoo robodoo added the r+ 👌 label Feb 14, 2020
@C3POdoo C3POdoo added the RD label Feb 14, 2020
@robodoo robodoo added the CI 🤖 label Feb 14, 2020
robodoo pushed a commit that referenced this pull request Feb 14, 2020
Since wizards shall not be unlinked by anyone, but as we want to get rid
of this specific one immediately to get rid of eventual credentials,
do it with elevated privileges.

Before this commit, the behavior was an authorization error when trying
to applying the payment method, in a fresh invoicing configuration.

closes #45363

X-original-commit: c8ee9c7
Signed-off-by: Martin Trigaux (mat) <mat@odoo.com>
@robodoo robodoo closed this Feb 14, 2020
@robodoo robodoo deployed to merge Feb 14, 2020 Active
VincentSchippefilt added a commit to odoo-dev/odoo that referenced this pull request Feb 18, 2020
Since wizards shall not be unlinked by anyone, but as we want to get rid
of this specific one immediately to get rid of eventual credentials,
do it with elevated privileges.

Before this commit, the behavior was an authorization error when trying
to applying the payment method, in a fresh invoicing configuration.

closes odoo#45363

X-original-commit: c8ee9c7
Signed-off-by: Martin Trigaux (mat) <mat@odoo.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

5 participants
You can’t perform that action at this time.