You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by GitBox <gi...@apache.org> on 2022/03/18 15:11:15 UTC

[GitHub] [airflow] omarsmak opened a new issue #22363: OpenAPI Specs misses a list of securitySchemes defined in 'security' which as result breaks generated OpenAPI client authentication mechanism

omarsmak opened a new issue #22363:
URL: https://github.com/apache/airflow/issues/22363


   ### Apache Airflow version
   
   2.2.4 (latest released)
   
   ### What happened
   
   Currently we have this definition in the OpenAPI specs:
   ```
     securitySchemes:
       Basic:
         type: http
         scheme: basic
       GoogleOpenId:
         type: openIdConnect
         openIdConnectUrl: https://accounts.google.com/.well-known/openid-configuration
       Kerberos:
         type: http
         scheme: negotiate
   
   # The API will provide support for plugins to support various authorization mechanisms.
   # Detailed information will be available in the plugin specification.
   security: []
   ```  
   
   When generating the REST APIs client through OpenAPI generator (Java client as an example which the one I am using), the client won't be able to authenticate since is missing the securitySchemes names in the `security: []` array. 
   
   If I include the securitySchemes property names in the `security` property like this:
   ```
   security:
     - Basic: []
     - GoogleOpenId: []
     - Kerberos: []
   ``` 
   This will work as expected.
   
   ### What you think should happen instead
   
   _No response_
   
   ### How to reproduce
   
   _No response_
   
   ### Operating System
   
   Linux
   
   ### Versions of Apache Airflow Providers
   
   _No response_
   
   ### Deployment
   
   Official Apache Airflow Helm Chart
   
   ### Deployment details
   
   _No response_
   
   ### Anything else
   
   If the maintainers thing this is a valid bug, I will be more than happy to send a PR :)
   
   ### Are you willing to submit PR?
   
   - [X] Yes I am willing to submit a PR!
   
   ### Code of Conduct
   
   - [X] I agree to follow this project's [Code of Conduct](https://github.com/apache/airflow/blob/main/CODE_OF_CONDUCT.md)
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [airflow] potiuk commented on issue #22363: OpenAPI Specs misses a list of securitySchemes defined in 'security' which as result breaks generated OpenAPI client authentication mechanism

Posted by GitBox <gi...@apache.org>.
potiuk commented on issue #22363:
URL: https://github.com/apache/airflow/issues/22363#issuecomment-1074180187


   Feel free to add it


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org