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
[API][AddressBook] Protect addresses with new firewall #11964
[API][AddressBook] Protect addresses with new firewall #11964
Conversation
AdamKasp
commented
Oct 27, 2020
Q | A |
---|---|
Branch? | master |
Bug fix? | no |
New feature? | yes |
BC breaks? | no |
License | MIT |
b298895
to
146244f
Compare
88290a7
to
8ae12f2
Compare
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.
There should be also scenarios for inability to add and delete an address from address book by a visitor
@@ -54,7 +54,7 @@ public function getItem(string $resourceClass, $id, string $operationName = null | |||
in_array('ROLE_USER', $user->getRoles(), true) && |
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.
It is not part of this PR but maybe you could change something so $user instanceof ShopUserInterface
won't be called 2 times? 🚀
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.
maybe :D but like you said it isn't the scope of this PR :)
7e47c65
to
e823246
Compare
...omer_account/address_book/preventing_not_logged_user_from_operations_on_address_book.feature
Outdated
Show resolved
Hide resolved
4234072
to
8f23152
Compare
8f23152
to
100f1da
Compare
100f1da
to
1c74e94
Compare
1c74e94
to
09ce746
Compare
09ce746
to
787d7e4
Compare
Thank you, Adam! 🎉 |