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.

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

Added 2 apis for streams #257

Open
wants to merge 2 commits into
base: streams
Choose a base branch
from
Open

Added 2 apis for streams #257

wants to merge 2 commits into from

Conversation

letzya
Copy link
Contributor

@letzya letzya commented Jun 4, 2024

Added 3 things:

  1. Added platform: linux/amd64 to the two external services petstore and httpbin - to solve the architecture error The requested image's platform (linux/amd64) does not match the detected host platform (linux/arm64)
    image
  2. Added API example post-to-ws - REST post to ws
  3. Added API example post-stream - post message to the tyk gateway log

@letzya letzya requested a review from davegarvey June 4, 2024 16:37
@letzya
Copy link
Contributor Author

letzya commented Jun 4, 2024

@davegarvey not sure why did got change deployments/tyk/data/tyk-dashboard/1/policies/policy-641c15dd0fffb800010197bf.json - any idea? I used the export script to dump the 2 APIs I created

@letzya letzya marked this pull request as ready for review June 4, 2024 16:45
@letzya
Copy link
Contributor Author

letzya commented Jun 4, 2024

FYI @dcs3spp @Keithwachira

@davegarvey
Copy link
Member

@davegarvey not sure why did got change deployments/tyk/data/tyk-dashboard/1/policies/policy-641c15dd0fffb800010197bf.json - any idea? I used the export script to dump the 2 APIs I created

It looks like the data just got shuffled around, rather than actually altered. I don't think the change will make any material difference.

@davegarvey
Copy link
Member

@letzya I don't see the new APIs you mention in the commits:

Added API example post-to-ws - REST post to ws
Added API example post-stream - post message to the tyk gateway log

It just looks like amends to the existing APIs.

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.

None yet

2 participants