Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Allow httpskill to accept optional headers #1690

Closed
sruthirameshv opened this issue Jun 23, 2023 · 3 comments
Closed

Allow httpskill to accept optional headers #1690

sruthirameshv opened this issue Jun 23, 2023 · 3 comments
Labels
core plugin Anything related to core plugins python Pull requests for the Python Semantic Kernel

Comments

@sruthirameshv
Copy link

sruthirameshv commented Jun 23, 2023 •
edited
Loading

Httpskill currently doesnt allow callers to add additional headers . For instance , if the user wants to pass bearer token to the post call ,
user should be able to pass the bearer token via context variable to the httpskill.

If this is an acceptable feature , I am happy to make the change.

In reference to now deleted issue: #1567

Tasks
Beta Give feedback
No tasks being tracked yet.
@alexchaomander
Copy link
Contributor

@shawncal @dluc @adrianwyatt Can you comment on this?

@alexchaomander alexchaomander added enhancement .NET Issue or Pull requests regarding .NET code labels Jun 26, 2023
@matthewbolanos matthewbolanos added the core plugin Anything related to core plugins label Jun 26, 2023
@alexchaomander alexchaomander added python Pull requests for the Python Semantic Kernel and removed .NET Issue or Pull requests regarding .NET code labels Jun 27, 2023
@awharrison-28 awharrison-28 moved this to Sprint: Planned in Semantic Kernel Jul 5, 2023
@nacharya1 nacharya1 removed the status in Semantic Kernel Aug 24, 2023
@nacharya1
Copy link
Contributor

@sruthirameshv - let me know if you are still interested to work on this? Would love to see a pr or contribution to #1567

@lemillermicrosoft
Copy link
Member

Closing as duplicate of #1567

@lemillermicrosoft lemillermicrosoft closed this as not planned Won't fix, can't repro, duplicate, stale Sep 8, 2023
@github-project-automation github-project-automation bot moved this to Sprint: Done in Semantic Kernel Sep 8, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
core plugin Anything related to core plugins python Pull requests for the Python Semantic Kernel
Projects
Semantic Kernel
Archived in project
Milestone
No milestone
Development

No branches or pull requests
5 participants
@matthewbolanos @alexchaomander @sruthirameshv @lemillermicrosoft @nacharya1