Skip to content

mime type fix for resource addition via mcp server #1366

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
Merged
merged 2 commits into from
Nov 4, 2025
Merged

mime type fix for resource addition via mcp server #1366

merged 2 commits into from
Nov 4, 2025
+25 −16

Conversation

@TS0713
Copy link
Collaborator

@TS0713 TS0713 commented Oct 29, 2025 •
edited
Loading

  • This PR addresses the issue 1381
  • mime type fix for adding resources via mcp server, added mime_type alias as mimeType in schemas > ResourceCreate
  • removed alembic folder created unexpectedly outside mcpgateway folder
  • This PR fixes mime type for resource which is added via mcp server

@TS0713 TS0713 requested a review from crivetimihai as a code owner October 29, 2025 18:20
@crivetimihai crivetimihai force-pushed the add_resource_via_mcpserver_fix branch from ea09a9c to 0d9470a Compare October 30, 2025 08:55
@crivetimihai crivetimihai mentioned this pull request Oct 30, 2025
Open
7 tasks
@TS0713 TS0713 force-pushed the add_resource_via_mcpserver_fix branch 2 times, most recently from e7d4a28 to 52f2a49 Compare November 3, 2025 08:31
@TS0713 TS0713 requested a review from crivetimihai November 3, 2025 09:02
TS0713 added 2 commits November 4, 2025 22:19
@TS0713 @crivetimihai
mime type fix for resource addition via mcp server
da03cc9 
Signed-off-by: Satya <tsp.0713@gmail.com>
@TS0713 @crivetimihai
added alias for mime_type in schemas > ResourceCreate, fixed pytest
1fe296a 
Signed-off-by: Satya <tsp.0713@gmail.com>
@crivetimihai crivetimihai force-pushed the add_resource_via_mcpserver_fix branch from 52f2a49 to 1fe296a Compare November 4, 2025 22:44
@crivetimihai crivetimihai merged commit 209d31d into IBM:main Nov 4, 2025
45 checks passed
@TS0713 TS0713 mentioned this pull request Nov 5, 2025
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@crivetimihai crivetimihai crivetimihai approved these changes

@madhav165 madhav165 Awaiting requested review from madhav165 madhav165 is a code owner

@kevalmahajan kevalmahajan Awaiting requested review from kevalmahajan kevalmahajan is a code owner

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@TS0713 @crivetimihai