Skip to content

.Net: Updates to SessionsPythonPlugin #11872

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Conversation

SergeyMenshykh
Copy link
Member

Motivation, Context and Description

This PR provides control over which domains requests can be sent to. Additionally, it moves functionality that is common to all operations of the plugin to one private method to remove duplication.

Contributes to: #10070

@SergeyMenshykh SergeyMenshykh requested a review from a team as a code owner May 2, 2025 18:01
@SergeyMenshykh SergeyMenshykh self-assigned this May 2, 2025
@markwallace-microsoft markwallace-microsoft added the .NET Issue or Pull requests regarding .NET code label May 2, 2025
@SergeyMenshykh SergeyMenshykh added the core plugin Anything related to core plugins label May 2, 2025
@SergeyMenshykh SergeyMenshykh moved this to Sprint: In Review in Semantic Kernel May 2, 2025
@SergeyMenshykh SergeyMenshykh added this pull request to the merge queue May 2, 2025
Merged via the queue into microsoft:main with commit e4804d5 May 2, 2025
24 checks passed
@SergeyMenshykh SergeyMenshykh deleted the add-allowed-domains-to-python-code-interpreter-plugin branch May 2, 2025 19:20
@github-project-automation github-project-automation bot moved this from Sprint: In Review to Sprint: Done in Semantic Kernel May 2, 2025
glorious-beard pushed a commit to glorious-beard/semantic-kernel that referenced this pull request May 6, 2025
### Motivation, Context and Description

This PR provides control over which domains requests can be sent to.
Additionally, it moves functionality that is common to all operations of
the plugin to one private method to remove duplication.

Contributes to:
microsoft#10070
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
core plugin Anything related to core plugins .NET Issue or Pull requests regarding .NET code
Projects
Status: Sprint: Done
Development

Successfully merging this pull request may close these issues.

4 participants