Skip to content

Can we enable discussions? Also, I'll push an HTTP transport fix #63

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
Open
rvoh-emccaleb opened this issue Jan 24, 2025 · 4 comments
Open

Can we enable discussions? Also, I'll push an HTTP transport fix #63

rvoh-emccaleb opened this issue Jan 24, 2025 · 4 comments

Comments

@rvoh-emccaleb
Copy link

rvoh-emccaleb commented Jan 24, 2025 •
edited
Loading

I've been trying to get the HTTP transports to work, and I have some incremental fixes I can push on Monday for a PR review-- some gotchas that appear to have slipped through the cracks.

But just to make sure -- the HTTP transports aren't working for anyone else, are they? (better for a discussion than an issue)
@rvoh-emccaleb rvoh-emccaleb changed the title Can we enable discussions? Can we enable discussions? Also, I'll push an HTTP transport fix Jan 24, 2025
rvoh-emccaleb added a commit to rvoh-emccaleb/mcp-golang that referenced this issue Jan 27, 2025
@rvoh-emccaleb
[metoro-io#63] Fix HTTP transport by removing duplication of base tra…
621a701 
…nsport
@rvoh-emccaleb rvoh-emccaleb mentioned this issue Jan 27, 2025
Closed
rvoh-emccaleb added a commit to rvoh-emccaleb/mcp-golang that referenced this issue Jan 27, 2025
@rvoh-emccaleb
Merge pull request #1 from rvoh-emccaleb/issue-63-http-transport
ff9812b 
[metoro-io#63] Fix HTTP transport by removing duplication of base transport
@artushin
Copy link

I haven't really tried but lmk if things are working for you and I'll give it a go in my implementation.

@rvoh-emccaleb
Copy link
Author

I haven't really tried but lmk if things are working for you and I'll give it a go in my implementation.

There's a proposed fix in an open PR.

@ecekyn
Copy link
Contributor

ecekyn commented Jan 30, 2025

Enabled discussions!

1 similar comment
@a67793581
Copy link

Enabled discussions!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@artushin @a67793581 @ecekyn @rvoh-emccaleb