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 2021/03/16 12:20:51 UTC

[GitHub] [airflow] jakubgs edited a comment on issue #14829: Removing a user and repeat OAuth login causes ERR_TOO_MANY_REDIRECTS

jakubgs edited a comment on issue #14829:
URL: https://github.com/apache/airflow/issues/14829#issuecomment-800209737


   Also, I identified why my user was getting the `Public` role. It's because I was using a custom `Dockerfile` to apply 3 fixes:
   ```Dockerfile
   FROM apache/airflow:2.0.1
   
   # We need this for GitHub OAuth via FAB
   RUN pip install authlib
   
   # HACK: Necessary before v2.0.2 is out.
   # https://github.com/apache/airflow/pull/14665
   RUN pip install flask-appbuilder==3.2.0
   
   # HACK: Fix for `Exception when importing` warnings.
   # https://github.com/apache/airflow/issues/14266
   RUN pip install -U azure-storage-blob apache-airflow-providers-microsoft-azure==1.1.0
   ```
   But using multiple `pip install` commands causes it to undo the installation of ``, so I've changed it to:
   ```Dockerfile
   FROM apache/airflow:2.0.1
   
   # WARNING: Has to be done as one command, or pip will uninstall packages.
   RUN pip install \
       # We need this for GitHub OAuth via FAB
       authlib \
       # HACK: Necessary before v2.0.2 is out.
       # https://github.com/apache/airflow/pull/14665
       flask-appbuilder==3.2.0 \
       # HACK: Fix for `Exception when importing` warnings.
       # https://github.com/apache/airflow/issues/14266
       azure-storage-blob apache-airflow-providers-microsoft-azure==1.1.0
   ```


----------------------------------------------------------------
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.

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