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

making Auth - bad clock errors clearer #87

Merged
merged 5 commits into from Aug 29, 2016

Conversation

Projects
None yet
4 participants
@jeremymv2
Contributor

jeremymv2 commented Aug 28, 2016

Description

Clarify 401 Auth - bad clock error.

[Describe what this change achieves]
This change adds clarification to the HTTP 401/403 Authentication errors due to time skew between client and server.

Issues Resolved

[List any existing issues this PR resolves]

Check List

**401, 403 Unauthorized - bad clock**
Occasionally, the system date/time will drift between client and server. If this drift is greate than a couple of minutes, the Chef Server will throw a 401 Aunauthorized and the request will not be forwarded to the Compliance server.

This comment has been minimized.

@chris-rock

chris-rock Aug 29, 2016

Collaborator

nit: unauthorized

when /403/
Chef::Log.error 'Possible offline Compliance Server or chef_gate auth issue.'
when /401|403/
Chef::Log.error 'auth issue: see audit cookbook TROUBLESHOOTING.md'

This comment has been minimized.

@chris-rock

chris-rock Aug 29, 2016

Collaborator

Nit: Auth issue:

@chris-rock

This comment has been minimized.

Collaborator

chris-rock commented Aug 29, 2016

Great idea to add a Troubleshoot.md @jeremymv2 I added to really minor comments. LGTM once they are fixed

@chris-rock chris-rock merged commit 8d4e201 into chef-cookbooks:master Aug 29, 2016

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details

@iennae iennae removed the in progress label Aug 29, 2016

@chris-rock

This comment has been minimized.

Collaborator

chris-rock commented Aug 29, 2016

Thanks @jeremymv2 for this clarification

@jeremymv2 jeremymv2 deleted the jeremymv2:time_drift_causes_401 branch Aug 29, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment