[openSUSE/open-build-service] 7ee8b8: Simplify ScmWebhookHeadersDataExtractor
Branch: refs/heads/master Home: https://github.com/openSUSE/open-build-service Commit: 7ee8b886592ff9358ad2e905e4fa1aefb5bcc6d5 https://github.com/openSUSE/open-build-service/commit/7ee8b886592ff9358ad2e9... Author: Henne Vogelsang <hvogel@opensuse.org> Date: 2024-04-10 (Wed, 10 Apr 2024) Changed paths: M src/api/app/controllers/concerns/scm_webhook_headers_data_extractor.rb M src/api/app/controllers/trigger_workflow_controller.rb Log Message: ----------- Simplify ScmWebhookHeadersDataExtractor No need to turn the request environment into instance variables in a before filter to use them in the same controller / request cycle. Commit: d9d3acb3c7bf9517cabfaa1876c077c9824a3be8 https://github.com/openSUSE/open-build-service/commit/d9d3acb3c7bf9517cabfaa... Author: Henne Vogelsang <hvogel@opensuse.org> Date: 2024-04-10 (Wed, 10 Apr 2024) Changed paths: M src/api/app/controllers/trigger_workflow_controller.rb Log Message: ----------- Move token type validaiton into it's own before filter This has nothing to do with creating a WorkflowRun... Commit: f5abeb15b3a7cb32d159b74a7f85adca455a5fd4 https://github.com/openSUSE/open-build-service/commit/f5abeb15b3a7cb32d159b7... Author: Henne Vogelsang <hvogel@opensuse.org> Date: 2024-04-10 (Wed, 10 Apr 2024) Changed paths: M src/api/app/controllers/trigger_workflow_controller.rb M src/api/app/services/trigger_controller_service/scm_extractor.rb M src/api/spec/controllers/trigger_workflow_controller_spec.rb Log Message: ----------- Disallow triggering for unknown SCM vendors Commit: 2b79c7555d624bae0142aa1ec4ee1993f7b01c1b https://github.com/openSUSE/open-build-service/commit/2b79c7555d624bae0142aa... Author: Henne Vogelsang <hvogel@opensuse.org> Date: 2024-04-10 (Wed, 10 Apr 2024) Changed paths: M src/api/app/controllers/concerns/scm_webhook_headers_data_extractor.rb M src/api/app/controllers/trigger_workflow_controller.rb M src/api/app/services/trigger_controller_service/scm_extractor.rb M src/api/spec/controllers/trigger_workflow_controller_spec.rb Log Message: ----------- Merge pull request #15958 from hennevogel/refactoring/TriggerWorkflowController Refactoring TriggerWorkflowController Compare: https://github.com/openSUSE/open-build-service/compare/7cccec6085ac...2b79c7... To unsubscribe from these emails, change your notification settings at https://github.com/openSUSE/open-build-service/settings/notifications
participants (1)
-
Henne Vogelsang