Skip to content

Conversation

@solvaholic
Copy link
Contributor

Lego does not support the _FILE variables for everything. For example:

AWS_ACCESS_KEY_ID Managed by the AWS client (AWS_ACCESS_KEY_ID_FILE is not supported)

https://go-acme.github.io/lego/dns/route53/

What does this PR do?

Fix statement about lego _FILE env var: s/Every/Many/ and instruct reader to follow linked documentation.

Motivation

I trusted the "every" in this doc so thoroughly I failed completely to notice the contradictory statement in lego's doc. I wasted most of a day chasing around route53: NoCredentialProviders. I disliked that, and I want future readers to have a better experience.

More

  • Added/updated tests
  • Added/updated documentation

Additional Notes

Traefik is amazing and I love the work y'all are doing!

I suspect the lack of support for AWS_ACCESS_KEY_ID_FILE has more to do with the AWS SDK than with lego or Traefik.

Copy link
Member

@juliens juliens left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Contributor

@ldez ldez left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks 👍

Copy link
Collaborator

@jbdoumenjou jbdoumenjou left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

solvaholic and others added 2 commits July 1, 2020 11:10
Lego does not support the `_FILE` variables for everything. For example:

> AWS_ACCESS_KEY_ID	Managed by the AWS client (AWS_ACCESS_KEY_ID_FILE is not supported)

https://go-acme.github.io/lego/dns/route53/
Co-authored-by: Roger D. Winans <solvaholic@users.noreply.github.com>
@solvaholic
Copy link
Contributor Author

Woohoo! Thanks y'all 🙇

@solvaholic solvaholic deleted the patch-1 branch July 1, 2020 13:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants