[obs-commits] [openSUSE/open-build-service] 05f3e3: [webui] Replace `require_login` with Pundit
Branch: refs/heads/master Home: https://github.com/openSUSE/open-build-service Commit: 05f3e31373f89457c363d3944741bc305ba9d8d0 https://github.com/openSUSE/open-build-service/commit/05f3e31373f89457c363d3... Author: Jacopo <beschi.jacopo@gmail.com> Date: 2020-09-30 (Wed, 30 Sep 2020) Changed paths: M src/api/app/controllers/concerns/webui/rescue_handler.rb M src/api/app/controllers/webui/cloud/azure/configurations_controller.rb M src/api/app/policies/application_policy.rb A src/api/app/policies/cloud/azure/configuration_policy.rb A src/api/config/initializers/patch_pundit_exception.rb R src/api/spec/cassettes/Webui_Cloud_Azure_ConfigurationsController/DELETE_destroy/1_3_1.yml R src/api/spec/cassettes/Webui_Cloud_Azure_ConfigurationsController/DELETE_destroy/1_3_2.yml A src/api/spec/cassettes/Webui_Cloud_Azure_ConfigurationsController/DELETE_destroy/for_logged_in_user/1_3_2_1.yml A src/api/spec/cassettes/Webui_Cloud_Azure_ConfigurationsController/DELETE_destroy/for_logged_in_user/1_3_2_2.yml R src/api/spec/cassettes/Webui_Cloud_Azure_ConfigurationsController/GET_show/with_Azure_configuration/1_1_2_1.yml A src/api/spec/cassettes/Webui_Cloud_Azure_ConfigurationsController/GET_show/with_Azure_configuration/1_1_3_1.yml M src/api/spec/cassettes/Webui_Cloud_Azure_ConfigurationsController/GET_show/without_Azure_configuration/creates_an_Azure_configuration.yml A src/api/spec/cassettes/Webui_Cloud_Azure_ConfigurationsController/PUT_update/behaves_like_require_logged_in_user/sets_flash_error_message.yml A src/api/spec/cassettes/Webui_Cloud_Azure_ConfigurationsController/PUT_update/behaves_like_require_logged_in_user/when_proxy_auth_mode_off/1_2_1_3_1.yml A src/api/spec/cassettes/Webui_Cloud_Azure_ConfigurationsController/PUT_update/behaves_like_require_logged_in_user/when_proxy_auth_mode_on/1_2_1_4_1.yml A src/api/spec/cassettes/Webui_Cloud_Azure_ConfigurationsController/PUT_update/behaves_like_require_logged_in_user/when_request_is_xhr/returns_Please_login_error.yml R src/api/spec/cassettes/Webui_Cloud_Azure_ConfigurationsController/PUT_update/with_invalid_parameters/1_2_2_1.yml A src/api/spec/cassettes/Webui_Cloud_Azure_ConfigurationsController/PUT_update/with_invalid_parameters/1_2_3_1.yml R src/api/spec/cassettes/Webui_Cloud_Azure_ConfigurationsController/PUT_update/with_valid_parameters/1_2_1_1.yml A src/api/spec/cassettes/Webui_Cloud_Azure_ConfigurationsController/PUT_update/with_valid_parameters/1_2_2_1.yml M src/api/spec/controllers/webui/cloud/azure/configurations_controller_spec.rb M src/api/spec/factories/azure_configuration.rb A src/api/spec/policies/cloud/azure/configuration_policy_spec.rb A src/api/spec/support/shared_examples/require_logged_in_user.rb Log Message: ----------- [webui] Replace `require_login` with Pundit This is a PR of a series which replaces `require_login` with `Pundit`. You can find further relevant info in #10083. Tackles Cloud::Azure::ConfigurationController. Ref #10083 Commit: c7c4f7dad17c57b2351b9da8b746b959f75c0420 https://github.com/openSUSE/open-build-service/commit/c7c4f7dad17c57b2351b9d... Author: Dany Marcoux <danymarcoux@gmail.com> Date: 2020-09-30 (Wed, 30 Sep 2020) Changed paths: M src/api/app/controllers/concerns/webui/rescue_handler.rb M src/api/app/controllers/webui/cloud/azure/configurations_controller.rb M src/api/app/policies/application_policy.rb A src/api/app/policies/cloud/azure/configuration_policy.rb A src/api/config/initializers/patch_pundit_exception.rb R src/api/spec/cassettes/Webui_Cloud_Azure_ConfigurationsController/DELETE_destroy/1_3_1.yml R src/api/spec/cassettes/Webui_Cloud_Azure_ConfigurationsController/DELETE_destroy/1_3_2.yml A src/api/spec/cassettes/Webui_Cloud_Azure_ConfigurationsController/DELETE_destroy/for_logged_in_user/1_3_2_1.yml A src/api/spec/cassettes/Webui_Cloud_Azure_ConfigurationsController/DELETE_destroy/for_logged_in_user/1_3_2_2.yml R src/api/spec/cassettes/Webui_Cloud_Azure_ConfigurationsController/GET_show/with_Azure_configuration/1_1_2_1.yml A src/api/spec/cassettes/Webui_Cloud_Azure_ConfigurationsController/GET_show/with_Azure_configuration/1_1_3_1.yml M src/api/spec/cassettes/Webui_Cloud_Azure_ConfigurationsController/GET_show/without_Azure_configuration/creates_an_Azure_configuration.yml A src/api/spec/cassettes/Webui_Cloud_Azure_ConfigurationsController/PUT_update/behaves_like_require_logged_in_user/sets_flash_error_message.yml A src/api/spec/cassettes/Webui_Cloud_Azure_ConfigurationsController/PUT_update/behaves_like_require_logged_in_user/when_proxy_auth_mode_off/1_2_1_3_1.yml A src/api/spec/cassettes/Webui_Cloud_Azure_ConfigurationsController/PUT_update/behaves_like_require_logged_in_user/when_proxy_auth_mode_on/1_2_1_4_1.yml A src/api/spec/cassettes/Webui_Cloud_Azure_ConfigurationsController/PUT_update/behaves_like_require_logged_in_user/when_request_is_xhr/returns_Please_login_error.yml R src/api/spec/cassettes/Webui_Cloud_Azure_ConfigurationsController/PUT_update/with_invalid_parameters/1_2_2_1.yml A src/api/spec/cassettes/Webui_Cloud_Azure_ConfigurationsController/PUT_update/with_invalid_parameters/1_2_3_1.yml R src/api/spec/cassettes/Webui_Cloud_Azure_ConfigurationsController/PUT_update/with_valid_parameters/1_2_1_1.yml A src/api/spec/cassettes/Webui_Cloud_Azure_ConfigurationsController/PUT_update/with_valid_parameters/1_2_2_1.yml M src/api/spec/controllers/webui/cloud/azure/configurations_controller_spec.rb M src/api/spec/factories/azure_configuration.rb A src/api/spec/policies/cloud/azure/configuration_policy_spec.rb A src/api/spec/support/shared_examples/require_logged_in_user.rb Log Message: ----------- Merge pull request #10200 from intrip/refactor_require_login_azure_configurations_ctrl Replace `require_login` with Pundit in Cloud::Azure::ConfigurationController Compare: https://github.com/openSUSE/open-build-service/compare/4595a6816af2...c7c4f7...
participants (1)
-
Dany Marcoux