TristanBilot / airflow-rbac-roles-cli

A tool to create Airflow RBAC roles with dag-level permissions from cli.
13 stars 6 forks source link

Unfortunately, script does not work #1

Open ghost opened 2 years ago

ghost commented 2 years ago

Hi, I have a Airflow installation using Docker on my Linux machine. It has username and password protected. There is no Google authentication used anywhere. When I run the script :

python3 rbac_roles_cli.py -u http://10.149.32.53:8081 -r NEW_PLATFORMING_ROLE -d IDS_UDL_S3_PLATFORMING_CASES

I get a connection error.

File "rbac_roles_cli.py", line 56, in create_rbac_role_with_permissions raise PermissionError(f"Error 401 returned, please check the access token if the page is protected by an authentication")

ghost commented 2 years ago

Even this link gives a permission error. This is my Airflow link.

http://10.149.32.53:8081/api/v1/roles

"detail": null, "status": 403, "title": "Forbidden", "type": "https://airflow.apache.org/docs/apache-airflow/2.3.3/stable-rest-api-ref.html#section/Errors/PermissionDenied" }

ghost commented 2 years ago

Finally, I think I could make this work !! I just had to add authetication with the API endpoint, which was missing earlier.

Must say your script to add roles & permissions via restAPI is really useful. Thank You !

TristanBilot commented 2 years ago

Hello there,

Thank you for using my module! How did you achieve to bypass your error? To which API did you authenticate? If your work could help other people, maybe you can open a pull request with your modifications?

Cheers, Tristan

On 11 Aug 2022, at 14:46, ghoshark @.***> wrote:



I think I could make this work !! I just had to add authetication with the API endpoint, which was missing earlier.

Must say your script to add roles & permissions via restAPI is really useful. Thanks!

— Reply to this email directly, view it on GitHubhttps://github.com/TristanBilot/airflow-rbac-roles-cli/issues/1#issuecomment-1211941354, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AJTYC3ZYL3OCTRGXYG44COTVYTYZZANCNFSM56H5HXVQ. You are receiving this because you are subscribed to this thread.Message ID: @.***>

elkan7 commented 1 year ago

Hello, I modified your code a bit so that it was authenticated by username and password, in addition to only adding a DAG from the list to the permissions. "it works for me"

How do I send you the file with the modifications for you to review?

Greetings

TristanBilot commented 1 year ago

Hey there, I appreciate your willing to contribute. Can you open a pull request where you apply your modifications? Do you know how to do that? Cheers

highway9ishika commented 3 days ago

Hi @elkan7, could you send me that modified file where you can create a role with all permissions

010NEHA commented 3 days ago

What permissions

On Mon, 14 Oct 2024, 11:44 Ishika Gupta, @.***> wrote:

Hi @elkan7 https://github.com/elkan7, could you send me that modified file where you can create a role with all permissions

— Reply to this email directly, view it on GitHub https://github.com/TristanBilot/airflow-rbac-roles-cli/issues/1#issuecomment-2410109295, or unsubscribe https://github.com/notifications/unsubscribe-auth/AXXPLHKYFFF25BYXWYEWG7LZ3NONHAVCNFSM6AAAAABP4IDPPWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDIMJQGEYDSMRZGU . You are receiving this because you were mentioned.Message ID: @.***>