Add Service ID to --verbose
output
#383
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
The
--verbose
flag prints the Fastly API token source as well as the Fastly API endpoint, but it would be helpful for users to also see the Service ID as this can be sourced from multiple locations:The motivation behind this change is that we received a report from a user to say they ran into an issue where the Service ID within the manifest file was being overridden by the
FASTLY_SERVICE_ID
environment variable. The root cause was that they had reused the same shell from an earlier test experiment and hadn't notice the environment variable being set.Adding the Service ID to the
--verbose
output should help users to pinpoint a possible issue like this.