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

feat(notes): images proxy #942

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

feat(notes): images proxy #942

wants to merge 2 commits into from

Conversation

n1rwana
Copy link
Member

@n1rwana n1rwana commented Aug 2, 2023

Этот PR добавляет прокси для изображений, который сейчас используется при отображении заметок, где пользователи могут встраивать любые изображения. Это решает проблему с различными IP-логгерами, так как теперь они получают только IP-адрес сервера, но не пользователя.

Также теперь ссылки в заметках ведут на /away.php?to=....

Resolves #916

image

@mrilyew
Copy link
Contributor

mrilyew commented Aug 3, 2023

Так нельзя же палить айпи сервера

@n1rwana
Copy link
Member Author

n1rwana commented Aug 3, 2023

Теперь можно включить/отключить замену изображений в заметках. Также, если она включена, возможно указать дополнительно сторонний прокси, например:

imagesProxy:
            replaceInNotes: true
            settings:
                url_prefix: "https://imgproxy-stag.sicepat.com/random43string/rs:fit:0:0:0/g:no/"
                base64_decode_url: false

в этом случае base64_decode_url = false, так как сторонний прокси принимает URL как раз закодированный в base64. по умолчанию этот параметр равен true

@WerySkok WerySkok requested a review from celestora August 9, 2023 13:03
@mrilyew mrilyew changed the title Безопасность заметок и прокси изображений feat(notes): images proxy Dec 11, 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
2 participants