Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: a couple of bugs in get_session_credentials #120

Merged
merged 1 commit into from
Aug 30, 2022

Conversation

paulfouquet
Copy link
Collaborator

@paulfouquet paulfouquet commented Aug 30, 2022

File "/app/scripts/aws/aws_helper.py", line 103, in get_session_credentials
    credentials: AwsFrozenCredentials = get_session(prefix).get_frozen_credentials()
AttributeError: 'Session' object has no attribute 'get_frozen_credentials'

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/app/scripts/non_visual_qa.py", line 161, in <module>
    main()
  File "/app/scripts/non_visual_qa.py", line 157, in main
    non_visual_qa(source)
  File "/app/scripts/non_visual_qa.py", line 145, in non_visual_qa
    file_check.run()
  File "/app/scripts/non_visual_qa.py", line 99, in run
    gdalinfo_process = run_gdal(gdalinfo_command, self.path)
  File "/app/scripts/gdal/gdal_helper.py", line 63, in run_gdal
    credentials = get_session_credentials(input_file)
  File "/app/scripts/aws/aws_helper.py", line 105, in get_session_credentials
    except client_sts.meta.client.exceptions.InvalidIdentityTokenException as e:
AttributeError: 'ClientMeta' object has no attribute 'client'

@paulfouquet paulfouquet requested a review from a team as a code owner August 30, 2022 01:35
@kodiakhq kodiakhq bot merged commit d6372e7 into master Aug 30, 2022
@kodiakhq kodiakhq bot deleted the fix/get-session-credentials-errors branch August 30, 2022 01:44
@github-actions github-actions bot mentioned this pull request Aug 30, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants