Branch: refs/heads/master
Home: https://github.com/openSUSE/open-build-service
Commit: 000e14cb3659871ad106e42115d23884b1e9a99e
https://github.com/openSUSE/open-build-service/commit/000e14cb3659871ad106e4...
Author: Adrian Schröter
Date: 2022-07-07 (Thu, 07 Jul 2022)
Changed paths:
M src/api/app/models/bs_request.rb
M src/api/app/models/bs_request/errors.rb
M src/api/app/models/bs_request_action.rb
M src/api/app/models/bs_request_action_add_role.rb
M src/api/app/models/bs_request_action_change_devel.rb
M src/api/app/models/bs_request_action_delete.rb
M src/api/app/models/bs_request_action_maintenance_incident.rb
M src/api/app/models/bs_request_action_maintenance_release.rb
M src/api/app/models/bs_request_action_release.rb
M src/api/app/models/bs_request_action_set_bugowner.rb
M src/api/app/models/bs_request_action_submit.rb
M src/api/test/functional/maintenance_test.rb
M src/api/test/functional/request_events_test.rb
Log Message:
-----------
[api] protect us against broken requests due to multiple entries
The active record validation is not protecting us here on creation
time, so introducing uniq_keys for validation.
Also we can conflict correctly between different action types this way.
Only action types which do not conflict with others for the same target
have therefore a prefix in the conlflict key.
Commit: c99822f7ad3afeb9cd70185e6612f9e8751c52af
https://github.com/openSUSE/open-build-service/commit/c99822f7ad3afeb9cd7018...
Author: Henne Vogelsang
Date: 2022-07-07 (Thu, 07 Jul 2022)
Changed paths:
M src/api/app/models/bs_request.rb
M src/api/spec/models/bs_request_action_spec.rb
Log Message:
-----------
Extract uniq action validation and adopt specs
Commit: 2963beb73aa41f58a63df595d3ea33bdfe6e2a8e
https://github.com/openSUSE/open-build-service/commit/2963beb73aa41f58a63df5...
Author: Henne Vogelsang
Date: 2022-07-20 (Wed, 20 Jul 2022)
Changed paths:
M src/api/app/models/bs_request.rb
M src/api/app/models/bs_request/errors.rb
M src/api/app/models/bs_request_action.rb
M src/api/app/models/bs_request_action_add_role.rb
M src/api/app/models/bs_request_action_change_devel.rb
M src/api/app/models/bs_request_action_delete.rb
M src/api/app/models/bs_request_action_maintenance_incident.rb
M src/api/app/models/bs_request_action_maintenance_release.rb
M src/api/app/models/bs_request_action_release.rb
M src/api/app/models/bs_request_action_set_bugowner.rb
M src/api/app/models/bs_request_action_submit.rb
M src/api/spec/models/bs_request_action_spec.rb
M src/api/test/functional/maintenance_test.rb
M src/api/test/functional/request_events_test.rb
Log Message:
-----------
Merge pull request #12570 from adrianschroeter/conflicting_action_detection
[api] protect us against broken requests due to multiple entries
Compare: https://github.com/openSUSE/open-build-service/compare/2bedc853072a...2963be...
Henne Vogelsang