search

zenden2k / image-uploader

A program for uploading images, screenshots and other files to a few dozen file hosting services. It has screenshooting capabilities and integrated image editor.
https://svistunov.dev/imageuploader
Apache License 2.0
117 stars 26 forks source link

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

Closed johnyxpro closed 1 year ago

johnyxpro commented 1 year ago

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

zenden2k commented 1 year ago

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

johnyxpro commented 1 year ago

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

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

stalkerok commented 1 year ago

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

johnyxpro commented 1 year ago

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

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

stalkerok commented 1 year ago

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

johnyxpro commented 1 year ago

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

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

stalkerok commented 1 year ago

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

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

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

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

johnyxpro commented 1 year ago

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

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

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

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

stalkerok commented 1 year ago

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

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

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

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

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

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