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.

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

Авторизация и прочие заголовки для зеркал #217

Merged
merged 7 commits into from
Jun 16, 2024
Merged

Авторизация и прочие заголовки для зеркал #217

merged 7 commits into from
Jun 16, 2024

Conversation

dmpas
Copy link
Member

@dmpas dmpas commented Jun 4, 2024

Добавлена возможность задавать произвольные заголовки для серверов в файле opm.cfg

@dmpas
oscript-library#216 Настройка запросов к хабу.
1c4d9e1 
Добавлены параметры передачи заголовка авторизации и дополнительных заголовков при получении пакетов с сервера.
EvilBeaver
EvilBeaver reviewed Jun 5, 2024
View reviewed changes
Copy link
Member

@EvilBeaver EvilBeaver left a comment

Choose a reason for hiding this comment

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

Еще же токен будет на операцию get, я верно понимаю?

src/cmd/Классы/КомандаOpm_Push.os Show resolved Hide resolved
@dmpas
Copy link
Member Author

dmpas commented Jun 5, 2024

Еще же токен будет на операцию get, я верно понимаю?

Собственно сейчас он на GET (и всё с ним связанное) и сделан.
"Ещё же токен будет на операцию" PUSH - с ней возюкаюсь.
Плюс думаю как учесть двухэтапную авторизацию: когда есть секрет и по нему надо получить краткосрочный токен.

@EvilBeaver
Copy link
Member

Плюс думаю как учесть двухэтапную авторизацию: когда есть секрет и по нему надо получить краткосрочный токен.

Разве нам такое надо? @nixel2007 что скажешь?

@dmpas
Copy link
Member Author

dmpas commented Jun 6, 2024

Плюс думаю как учесть двухэтапную авторизацию: когда есть секрет и по нему надо получить краткосрочный токен.

Разве нам такое надо? @nixel2007 что скажешь?

@EvilBeaver я для тестов набросал себе на коленке хаб на (тададаам) Элементе. А там авторизация двухэтапная. Вот я и решил заморочитьтся...

@EvilBeaver
Copy link
Member

@EvilBeaver я для тестов набросал себе на коленке хаб на (тададаам) Элементе. А там авторизация двухэтапная. Вот я и решил заморочитьтся...

Это кстати очень прикольно. Я начинал, но забросил. Но сейчас gitflic будет повторять протокол между opm и нашим хабом и на первом этапе достаточно этого

@dmpas
Copy link
Member Author

dmpas commented Jun 6, 2024

и на первом этапе достаточно этого

ну тогда буду закругляться.

@dmpas dmpas changed the title [WIP] Авторизация и прочие заголовки для зеркал Авторизация и прочие заголовки для зеркал Jun 10, 2024
@dmpas dmpas requested a review from EvilBeaver June 10, 2024 10:06
@nixel2007
Copy link
Member

Таймаут бы в пример cfg, а так вроде все хорошо!

@EvilBeaver EvilBeaver merged commit d3eb457 into oscript-library:master Jun 16, 2024
@nixel2007 nixel2007 linked an issue Jun 16, 2024 that may be closed by this pull request
Реестр пакетов GitFlic. Добавить авторизационный токен #216
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@EvilBeaver EvilBeaver EvilBeaver approved these changes

@nixel2007 nixel2007 nixel2007 approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Реестр пакетов GitFlic. Добавить авторизационный токен
3 participants
@dmpas @EvilBeaver @nixel2007