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

SEP-38: make SEP-10 token optional for some endpoints #1144

Merged
merged 4 commits into from
Mar 8, 2022

Conversation

marcelosalloum
Copy link
Contributor

@marcelosalloum marcelosalloum commented Mar 8, 2022

What

Make SEP-10 token optional for the endpoints:

  • GET /info
  • GET /price
  • GET /prices

Why

Client applications may want to show rates from multiple liquidity providers to a user, or simply display the exchange rates of an anchor's pairs before the user has indicated they want to start a transaction.

Closes #1086.

Copy link
Member

@leighmcculloch leighmcculloch left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks great, I think this change is good and very backwards compatible.

I make a couple suggestions below.

Could you also update the version by bumping the minor version number, and the updated field in the preamble?

ecosystem/sep-0038.md Outdated Show resolved Hide resolved
ecosystem/sep-0038.md Outdated Show resolved Hide resolved
JakeUrban
JakeUrban previously approved these changes Mar 8, 2022
Copy link
Contributor

@JakeUrban JakeUrban left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I agree with Leigh's points, but otherwise LGTM! Thanks for taking this on.

marcelosalloum and others added 3 commits March 8, 2022 14:25
Co-authored-by: Leigh McCulloch <351529+leighmcculloch@users.noreply.github.com>
@marcelosalloum marcelosalloum merged commit cc75223 into master Mar 8, 2022
@marcelosalloum marcelosalloum deleted the sep38-optional-auth branch March 8, 2022 17:45
marcelosalloum added a commit to stellar/stellar-anchor-tests that referenced this pull request Mar 29, 2022
marcelosalloum added a commit to stellar/stellar-anchor-tests that referenced this pull request Mar 30, 2022
…-38 (#79)

### What

Update tests to make it compliant with the new version of [SEP-38](https://stellar.org/protocol/sep-38):
* `QUOTE_SERVER` is actually meant to be `ANCHOR_QUOTE_SERVER`.
* Since [stellar-protocol#1144](stellar/stellar-protocol#1144), only the `/quote(s)` endpoints require SEP-10 authentication.
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.

SEP-38: Allow unauthenticated requests to non-/quote endpoints
3 participants