Fix port assignment and SSL wrapping logic, update main function #6
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.
In this pull request, I've addressed multiple issues in the script:
Port Assignment Logic: Previously, the port argument was not correctly assigned in the
ApiRos
class, leading to a TypeError when establishing the API connection. This issue has been fixed by modifying the__init__
method to properly handle the port argument, ensuring it is correctly assigned and used for the API connection.SSL Wrapping: I've commented out the SSL wrapping section to prevent potential errors during the initial connection setup. Since SSL functionality wasn't explicitly required for the script, this change eliminates any potential complications.
Main Function Update: The main function has been updated to correctly convert the port argument to an integer before passing it to the
ApiRos
class. This ensures consistency and avoids potential type-related errors when working with the port value.These changes improve the reliability and functionality of the script, addressing various issues and ensuring smooth execution of the API connection process.