Skip to content

Graph: add ssl=True by default#180

Merged
cyc60 merged 3 commits intomasterfrom
graph-ssl
Feb 5, 2026
Merged

Graph: add ssl=True by default#180
cyc60 merged 3 commits intomasterfrom
graph-ssl

Conversation

@cyc60
Copy link
Contributor

@cyc60 cyc60 commented Feb 5, 2026

No description provided.

Copilot AI review requested due to automatic review settings February 5, 2026 13:53
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR adds SSL support to the Graph client by introducing an optional ssl parameter that defaults to True. This ensures secure connections are enabled by default for GraphQL transport operations while maintaining backward compatibility through the optional parameter.

Changes:

  • Added ssl parameter to the Graph client constructor with a default value of True
  • Updated the AIOHTTPTransport instantiation to include the SSL configuration
  • Bumped the package version from v0.12.7 to v0.12.8

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
sw_utils/graph/client.py Added ssl parameter to constructor and passed it to AIOHTTPTransport for secure connections
pyproject.toml Incremented version number to reflect the new feature addition

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copilot AI review requested due to automatic review settings February 5, 2026 14:20
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@cyc60 cyc60 merged commit 3b127f8 into master Feb 5, 2026
3 checks passed
@cyc60 cyc60 deleted the graph-ssl branch February 5, 2026 14:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants