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

Add support for ${DOWNLOAD_URI} #29

Closed
wants to merge 1 commit into from
Closed

Conversation

Jipok
Copy link

@Jipok Jipok commented Feb 14, 2024

Позволит заменять адрес сервера. Полезно для локального использования:
bookReadLink": "http://mydomain.ru/#/reader?url=http://127.0.0.1:8086${DOWNLOAD_URI}

Так как я использую Jauth а клиент не передаёт свои куки при запросе, получается что клиент не может скачать книгу с сервера(т.к. там требуется авторизация).

@bookpauk bookpauk closed this Mar 25, 2024
@Jipok
Copy link
Author

Jipok commented Mar 25, 2024

А можно какой-нибудь комментарий по отказу?
Помимо того что я не обратил внимания на то что автоформатирование весь файл поменяло, в чём проблема?

@bookpauk
Copy link
Owner

bookpauk commented Mar 25, 2024

Простите, устал, сходу не разобрался, для чего это нужно. Берем в работу, с сохранением возможности предыдущего ${DOWNLOAD_LINK}. Если указан DOWNLOAD_URI - забираем только параметры, без хоста. Мержить не буду, доработаю руками.

@bookpauk bookpauk reopened this Mar 25, 2024
bookpauk added a commit that referenced this pull request Apr 4, 2024
@bookpauk bookpauk closed this Apr 4, 2024
@Jipok
Copy link
Author

Jipok commented Apr 5, 2024

Спасибо.

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