WIP - Dependency Updates and Compatibility Issues #24
Conversation
This file contains 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
…, Hyper, rustls, and other dependency etc.)
|
@nitinawari I'm happy to help debug this! Currently it looks like there are thousands of lines of diffs that don't actually change anything, making it next to impossible for me to see what you're actually changing. Could you clean that up? After that I'll take a look! |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in 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.
This pull request is focused on updating various dependencies within the project, specifically http-body, http-body-util, and Rust to the latest versions. Key changes include:
Dependency Updates:
Replaced http-body with http-body-util.
Updated code to accommodate new Rust version requirements.
Adjusted struct paths and incorporated necessary changes for rustls.
Issues Encountered:
Hyper Library: Significant changes in the Hyper library have caused compatibility issues. Many functionalities related to HTTPS server connections have been altered or removed.
Feature Removal: rustls has removed or relocated some features, affecting imports and implementations.
Current Status:
The code builds successfully but has issues with HTTPS connectivity and Hyper-related functionalities.
Changes have been made to align with the new API structures, but the implementation is incomplete and requires further validation.
Action Needed:
Hyper Functionality: Guidance on understanding the new Hyper API or suggestions for alternatives.
Feature Changes: Assistance with adapting to removed features in rustls or finding equivalent functionality.
Next Steps:
Review the current state of the PR and provide feedback on resolving the issues mentioned.
Suggest any further modifications or alternatives to address compatibility problems.
Branch Information:
Branch: dependency-update-in-progress
Additional Notes:
This PR is a work-in-progress and is not yet ready for merging. It is intended to share the current progress and seek assistance in overcoming the listed issues.