Branch: refs/heads/master
Home: https://github.com/openSUSE/open-build-service
Commit: 8a08683be8bbf7e270c4b68b6280eaa4bbf3a2d8
https://github.com/openSUSE/open-build-service/commit/8a08683be8bbf7e270c4b6...
Author: Christian Bruckmayer
Date: 2015-10-23 (Fri, 23 Oct 2015)
Changed paths:
M src/api/app/models/bs_request.rb
Log Message:
-----------
[webui] Move request permission check in lock statement
User reported that it was possible to accept a request several times.
Changing the request state is already wrapped in a transaction but
checking the permissions and creating the history is not. So finally
moved permission check and creating history also in the with_lock
block to avoid race conditions. This will fix #1157.
Commit: da9ec4a61581c436fe10d7133125d9109f1778f6
https://github.com/openSUSE/open-build-service/commit/da9ec4a61581c436fe10d7...
Author: Henne Vogelsang
Date: 2015-10-23 (Fri, 23 Oct 2015)
Changed paths:
M src/api/app/models/bs_request.rb
Log Message:
-----------
Merge pull request #1297 from ChrisBr/request_double_accept
[webui] Move request permission check in lock statement
Compare: https://github.com/openSUSE/open-build-service/compare/967be8e774c4...da9ec4...