Skip to content

Made tokens external to allow parallelism and CI updated#4

Closed
rawe0 wants to merge 1 commit into
mainfrom
dev
Closed

Made tokens external to allow parallelism and CI updated#4
rawe0 wants to merge 1 commit into
mainfrom
dev

Conversation

@rawe0

@rawe0 rawe0 commented Aug 28, 2024

Copy link
Copy Markdown
Member

Made tokens external to allow parallelism and CI has been updated to build the pip package and install it. This allows us to remove the "magic" (adding the src path to the unit tests).

README, examples and tests have been updated to reflect the requirements the external handling of the token imposes.

@rawe0 rawe0 closed this Aug 28, 2024
@rawe0 rawe0 deleted the dev branch August 28, 2024 16:21
Comment thread examples/auth_example.py
client.auth.set_client_secret("<your_client_secret_here>")
# Checks if token is expired if not return ('x-access-token', <token>)
# if expired, request a new token from reva
auth_token = client.auth.get_token()

Check warning

Code scanning / CodeQL

Variable defined multiple times

This assignment to 'auth_token' is unnecessary as it is [redefined](1) before this value is used.
text_resource = Resource.from_file_ref_and_endpoint("/eos/user/r/rwelande/text_file.txt")
res = client.file.touch_file(touch_resource)
res = client.file.touch_file(text_resource)
res = client.file.touch_file(client.auth.get_token(), touch_resource)

Check warning

Code scanning / CodeQL

Variable defined multiple times

This assignment to 'res' is unnecessary as it is [redefined](1) before this value is used.
print(res)

res = client.file.touch_file(touch_resource)
res = client.file.touch_file(client.auth.get_token(), touch_resource)

Check warning

Code scanning / CodeQL

Variable defined multiple times

This assignment to 'res' is unnecessary as it is [redefined](1) before this value is used.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants