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

не актуальный домен #2

Open
centralhardware opened this issue Jun 15, 2022 · 5 comments
Open

не актуальный домен #2

centralhardware opened this issue Jun 15, 2022 · 5 comments

Comments

@centralhardware
Copy link

habra переехала на другой домен. На старом только редирект на новый.

@istudyatuni
Copy link
Owner

А какой именно новый и какой старый домен? В коде прописан редирект на geekr.vercel.app

@LencoDigitexer
Copy link

#3 - сделал пул реквест. Желательно изменить название репозитория и сделать запрос на изменение расширения в https://github.com/jarvis394/geekr/#extensions

@istudyatuni
Copy link
Owner

istudyatuni commented Aug 20, 2022

А, понял. Я не изменил на geekr потому что там есть свой редирект, хотя с другой стороны согласен, не очень удобно ждать 2 редиректа (или там уже больше получается)

Upd. Я запутался, но сейчас понял. Редирект прописан:

let habra = '/geekr.vercel.app/post/$2/'

В regex.post лежит regex, который определяет, с какой страницы необходимо редиректить. Про то, почему не редиректит нормально с habra.js.org, написано в readme (первый абзац курсивом). Соответственно не понятно, не редиректит нормально с обычного хабра, или откуда?

@LencoDigitexer
Copy link

Автор habra. изменил брендинг. Теперь это geekr.
Соответственно, он поменял домен на geekr.vercel.app
Если перейти на habra.js.org, то будет два перенаправления - на habra.vercel.app и в конце всё равно будет geekr.vercel.app

Нужно поменять домены здесь на geekr.vercel.app:

post: /\/(m?\.?habr\.com|habra\.js\.org)\/.+\/([0-9]{1,})(\/?.{1,})?/,

post: /\/(m?\.?habr\.com|geekr\.vercel\.app)\/.+\/([0-9]{1,})(\/?.{1,})?/,

urls: ["*://*.habr.com/*", "*://*.habra.js.org/*"],

urls: ["*://*.habr.com/*", "*://*.geekr.vercel.app/*"],

"permissions": ["webRequest", "*://*.habr.com/*", "*://*.habra.js.org/*", "webRequestBlocking", "storage"],

"permissions": ["webRequest", "*://*.habr.com/*", "*://*.geekr.vercel.app/*", "webRequestBlocking", "storage"],

Про то, почему не редиректит нормально с habra.js.org, написано в readme (первый абзац курсивом).

Я не изменил на geekr потому что там есть свой редирект

Это костыли, все равно будет перенаправления, что очень неудобно. Да и чисто логически будет лучше, если расширение будет сразу делать своё дело - перенаправлять на оригинальный хост теперь уже geekr.

@istudyatuni
Copy link
Owner

Итак, во-первых, я поправил себя, что "не изменил на geekr потому что там есть свой редирект", потому что редирект прописан. Во-вторых, regex.post отвечает за проверку, надо ли редиректить с текущего адреса. В-третьих, если идет редирект с habra.js.org на habra.vercel.app, значит расширение отключено

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

No branches or pull requests

3 participants