While there is a link to /oauth2 on the /oauth page, landing on /oauth2 right
away makes it obvious to server admins that /oauth2 is where they should be.
Discussion: <https://lists.sr.ht/~sircmpwn/sr.ht-discuss/%3C877cwnhpni.fsf%40florness.com%3E>
Signed-off-by: David Florness <david@florness.com>
---
config.example.ini | 2 +-gitsrht/default_query.graphql | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/config.example.ini b/config.example.ini
index 7c645e8..beface9 100644
--- a/config.example.ini+++ b/config.example.ini
@@ -112,7 +112,7 @@ webhooks=redis://localhost:6379/1
post-update-script=/usr/bin/gitsrht-update-hook
#
# git.sr.ht's OAuth client ID and secret for meta.sr.ht
-# Register your client at meta.example.org/oauth+# Register your client at meta.example.org/oauth2oauth-client-id=CHANGEME
oauth-client-secret=CHANGEME
#
diff --git a/gitsrht/default_query.graphql b/gitsrht/default_query.graphql
index 0ddaa07..630fffb 100644
--- a/gitsrht/default_query.graphql+++ b/gitsrht/default_query.graphql
@@ -30,7 +30,7 @@ query {
# On this page, you have been automatically authorized to make API requests
# with your sr.ht login cookie. If you wish to make GraphQL requests outside
- # of the browser, create a personal access token at https://meta.sr.ht/oauth+ # of the browser, create a personal access token at https://meta.sr.ht/oauth2 #
# curl \
# -H Authorization:"Bearer <your oauth token>" \
--
2.39.1
I think we can skip this for config.example.ini; we mostly use internal
auth now and will be remove these config details in the foreseeable
future in any case.
default_query.graphql should be updated, though, and the update looks
good for all affected services if you'd like to prepare patches for
them.
BTW, there are other repos that could also get a patch for this, but I
was gonna hold off on sending those until this one gets accepted (in
case folks disagree with the motivation).
[PATCH git.sr.ht v2] Replace meta.sr.ht's /oauth endpoint with /oauth2 in comment
Export this patch
While there is a link to /oauth2 on the /oauth page, landing on /oauth2 right
away makes it obvious to server admins that /oauth2 is where they should be.
Discussion: <https://lists.sr.ht/~sircmpwn/sr.ht-discuss/%3C877cwnhpni.fsf%40florness.com%3E>
Signed-off-by: David Florness <david@florness.com>
---
v1 -> v2:
- ignore /oauth in config.example.ini: <https://lists.sr.ht/~sircmpwn/sr.ht-discuss/%3C877cwnhpni.fsf%40florness.com%3E#%3CCQI5AI6ADO6H.39CESCLMYYHS5@taiga%3E>
gitsrht/default_query.graphql | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/gitsrht/default_query.graphql b/gitsrht/default_query.graphql
index 0ddaa07..630fffb 100644
--- a/gitsrht/default_query.graphql+++ b/gitsrht/default_query.graphql
@@ -30,7 +30,7 @@ query {
# On this page, you have been automatically authorized to make API requests
# with your sr.ht login cookie. If you wish to make GraphQL requests outside
- # of the browser, create a personal access token at https://meta.sr.ht/oauth+ # of the browser, create a personal access token at https://meta.sr.ht/oauth2 #
# curl \
# -H Authorization:"Bearer <your oauth token>" \
--
2.39.2