fix: Modify the status code detection range of the check function to 2xx#161
Open
zhangxinhui02 wants to merge 1 commit intoezhov-evgeny:developfrom
Open
fix: Modify the status code detection range of the check function to 2xx#161zhangxinhui02 wants to merge 1 commit intoezhov-evgeny:developfrom
check function to 2xx#161zhangxinhui02 wants to merge 1 commit intoezhov-evgeny:developfrom
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
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.
Hello! I recently used webdav-client-python-3 to connect to my WebDAV service provider, but I always encounter issues when creating new folders.
After tracing and debugging in the IDE, I found that the
Client.check()function checks the server status before executing modifications. TheClient.check()function considers a 200 status code as correct, but my WebDAV service provider returns a 207 status code, which should also be considered correct. In fact, all 2xx status codes should be considered correct.I made some simple modifications to the code, enabling the
Client.check()function to correctly handle all 2xx status codes.