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

Fixed imageban.ru (only registered users can upload images) #359

Merged
merged 2 commits into from
Jul 2, 2023

Conversation

johnyxpro
Copy link
Contributor

Fixed imageban.ru (only registered users can upload images)

@zenden2k
Copy link
Owner

zenden2k commented Jun 30, 2023

Почему решили отказаться от использования API?
Пусть пользователи получают Secret key здесь и вписывают его в программу.
Я стремлюсь не хранить пароль пользователя, если это не требуется.

@johnyxpro
Copy link
Contributor Author

Почему решили отказаться от использования API? Пусть пользователи получают Secret key здесь и вписывают его в программу. Я стремлюсь не хранить пароль пользователя, если это не требуется.

  1. У api есть лимиты загрузок.
  2. Через Client ID и Secret key работа организована очень неудобно. Большинство пользователей даже не поймут что это за параметры такие в программе и зачем они нужны.
    В программе нет подсказок и удобного мануала касательно этих параметров. Для большинства юзеров в текущем виде - просто нерабочий imageban в вашей проге.
    Если хотите чтобы работа была обязательно через эти параметры и главное чтобы для пользователя было удобно - можно запрашивать у юзера единоразово логин-пароль, программа просто логинится на imageban (не сохраняя логин-пасс локально), берёт с imageban эти 2 параметра и уже затем их сохраняет и далее с ними работает. Тогда для юзера будет всё понятно и просто, а под капотом проги никаких сохранений пароля.
    Правда реализация сего в проге будет сложной.
    Поэтому легче уж сохранять логин-пароль. И работа с ним легче - благо есть отдельное окошко для запроса их у юзера.
    В моей реализации, через логин-пароль для большинства юзеров будет сходу понятно что и как и у них заработает imageban.
    При желании, можете сохранять локально все логин-пароли шифровано.
    Также есть предложение - сохранять пароль локально в виде md5, и затем работать с ним, благо отправлять на сервер нужно именно в таком виде.

@zenden2k zenden2k merged commit 729bad9 into zenden2k:master Jul 2, 2023
@stalkerok
Copy link
Contributor

@johnyxpro, а зачем вы удалили nm.addQueryParam("inf", "1"); ?
Верните обратно! Иначе придётся самому возвращать её на место.

@johnyxpro
Copy link
Contributor Author

@johnyxpro, а зачем вы удалили nm.addQueryParam("inf", "1"); ? Верните обратно! Иначе придётся самому возвращать её на место.

на тот момент когда я переделывал этот модуль imageban, на сайте imageban этот параметр не использовался.
Я разобрал передачу всех параметров этим сайтом, удалил всё что было раньше и вставил только параметры которые использовались imageban'ом на то время. Если сравнивать параметры с тем что было, то получится что некоторые парам-ы удалил, а некоторые добавил новые которых даже не было. Но на деле, все параметры я добавлял с чистого листа.

@stalkerok
Copy link
Contributor

В любом случае, это нужный параметр, строку нужно вернуть на место.

@johnyxpro
Copy link
Contributor Author

johnyxpro commented Aug 13, 2023

В любом случае, это нужный параметр, строку нужно вернуть на место.

Можете пояснить зачем он нужен? Без него всё прекрасно работало.
И, если добавление этого параметра действительно необходимо и оправдано, то добавляйте.

@stalkerok
Copy link
Contributor

Можете пояснить зачем он нужен? Без него всё прекрасно работало.

И работает. Это информация на превью, вес и ширина/высота. Без этого параметра у вас будут голые серверные миниатюры.

И, если добавление этого параметра действительно необходимо и оправдано, то добавляйте.

У вас остался форк, и это вы внесли эти изменения.

@johnyxpro
Copy link
Contributor Author

Это информация на превью, вес и ширина/высота. Без этого параметра у вас будут голые серверные миниатюры.

Не всем нужна эта инфа на превью.
В таком случае пусть автор решает.
Я бы предложил автору в самой программе сделать настройку галочку (если её нет) с запросом нужна ли инфа на превью. И если юзер отмечает её, тогда отсылать этот параметр на сервер imageban.

У вас остался форк, и это вы внесли эти изменения.

Если вам прям лично очень нужна инфа на превью, вы можете у себя в папке приложения, в файле /Data/Scripts/imageban.nut
добавить эту строку с параметром и у вас будет эта инфа на превью.

@stalkerok
Copy link
Contributor

Не всем нужна эта инфа на превью.

Большинству. По этому в проге есть свои собственные миниатюры.

Я бы предложил автору в самой программе сделать настройку галочку (если её нет) с запросом нужна ли инфа на превью.

Вот с этим согласен. В гуи должны быть эти настройки.

Если вам прям лично очень нужна инфа на превью

Я настроил под себя все что нужно сразу после обновления.

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.

3 participants