generated from deploymenttheory/Template
-
Notifications
You must be signed in to change notification settings - Fork 0
rebase dev branch #77
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Merge pull request #4 from deploymenttheory/main
Refactor Config struct in httpclient_client.go
Added func DetermineAuthMethod with auth type validation
Refactor client secret validation regex pattern
Update ClientOptions struct and add helper function for redacting sen…
Update Authorization header in httpclient_request.go
Refactor logger package and update logger configuration
Refactor error handling and add support for extracting multiple HTML …
Handle unmarshaling errors and log HTML error messages
Refactor error handling in JamfPro API handler
Add logger to JamfAPIHandler constructor
Refactor processResponse function to handle API errors and log responses
Refactor error handling in processResponse function
Refactor response processing in httpclient_request.go
Fix response body closing in HTTP client requests
Fix response body resource leak
Refactor DoRequestV2 to DoRequest and add DoMultipartRequest
Refactor error handling in JamfAPIHandler
Refactor error handling in HTTP client
Bumps [go.uber.org/zap](https://github.com/uber-go/zap) from 1.26.0 to 1.27.0. - [Release notes](https://github.com/uber-go/zap/releases) - [Changelog](https://github.com/uber-go/zap/blob/master/CHANGELOG.md) - [Commits](uber-go/zap@v1.26.0...v1.27.0) --- updated-dependencies: - dependency-name: go.uber.org/zap dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com>
Update GitHub Actions workflow trigger
Update logger tests to use expected message variables
Refactor logger tests and add sub-test for Fatal method
Adjust delta for wait duration in rate limit handler test
Refactor rate limit header parsing and add test cases***
Add skew buffer to expected wait time in rate limit test
Refactor backoff calculation in httpclient_rate_handler_test.go
Refactor logger tests and remove commented code
Remove unnecessary import and commented code
Add API handler methods to MockAPIHandler
Fix console encoding separator in logger configuration
….uber.org/zap-1.27.0 Bump go.uber.org/zap from 1.26.0 to 1.27.0
ShocOne
added a commit
that referenced
this pull request
Feb 23, 2024
Merge pull request #77 from deploymenttheory/main
# for free
to join this conversation on GitHub.
Already have an account?
# to comment
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.
Change
Type of Change
Please DELETE options that are not relevant.
Checklist