-
Notifications
You must be signed in to change notification settings - Fork 14
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
Unable to share with Shaarlier for Android + [resolved] Shaarli UI is broken, CSS/JS not loading (Nginx issue ?) #75
Comments
Hum, that's a pitty… Are you able to check fail2ban logs ? If you enter a wrong password for instance, does it behaves differently ? I wonder at which steps it fails (client side, server side, fail2ban side…) |
hello, |
Yeah that version is probably here since the last (or first 🤔) patch I made… and not usable anymore. I'm a bit lost about where you finally ended up… your current Shaarli UI is broken, that the issue ? |
To summarise, Shaarlier does not work with the API code but only with the ID/password. |
Ok, thank you.
Ok, then it's definitely something that is Yunohost specific. Thank you for testing this ! About the broken layout : it appeared after an upgrade ? It sounds like some CSS is not working/loaded… I don't have it on a 0.12.1~ynh2 version, so I wonder what specific… thing made it fail on your side… 🤔 |
So i try to install Shaarli 0.12.1~ynh2 and i chose the "public" mode. |
Thank you for the feedback. I have to say I am lost… maybe someone from @YunoHost-Apps/apps-group will have some clue… |
what about allowing access to the API part (I am just guessing here) |
That's what I first thought, be @pp-r tried in public mode, so the API shouldn't be blocked 🤔 |
Well just for the sack of testing I made a branch with API permission : https://github.com/YunoHost-Apps/shaarli_ynh/tree/API |
@pp-r : |
Hello, |
That would mean this is broken since January… 😱 So that has something to do with this commit bb64f6820c701f0dbcec4c72176141618b0a6c24… How long have you been using this Shaarli ? Before January 2021 ? That would mean that there is an issue with the template used. I'll try to compare it with mine, to see if there is any difference… Also, maybe you can try to copy that /tlp folder from your Custom WebApp Shaarli to Yunohost one (you might need to change the owner after that, I believe the command is chown shaarli /var/www/shaarli/tpl/). |
I don't use Shaarli but i just want to try it.
cd /var/www/shaarli/tpl ls /var/www/shaarli/tpl/* -l
cd /var/www/shaarli/tpl/ ls /var/www/shaarli/tpl/* -l Edit : |
In fact I needed the file list, to 1) check if all permissions where good 2) check if all files had the same size as mines (a quick & dirty wait to check if they are the same) 😅 Ok so I installed latest version on a test server… same CSS issue. Hey @YunoHost-Apps/apps-group : have you any idea how to fix that issue (without creating a security flaws) ? 🙏
|
So, i've reinstall in 2 times ("old" install + upgrade API branch) in order to have not broken css layout. /var/www/shaarli/tpl/vintage: |
Thank you for sharing the full result. I don't see anything special. Maybe I need the content of
So in that case it works ?! |
In the case of it works : /var/www/shaarli/tpl/default/js$ ls -l I can only make it work by this command |
Ok, so we have the some content in those folders. Then I have no clue. I don't see what changed since that version (the particular commit) you're pointing… Are you sure the next commit (https://github.com/YunoHost-Apps/shaarli_ynh/tree/bb64f6820c701f0dbcec4c72176141618b0a6c24) don't work ? |
So i've uninstall then reinstall from your link |
Great, so we're making progress :) About the API, I have no idea, sorry :/ But by testing each commit one by one, we may find where was the issue… don't do it, I'll handle that :) |
#77 should fix the broken User Interface problem :) |
Can you confirm that this change fix that broken UI issue ? |
Hello, I confirm that now installation work and the UI is now ok. Sharing with Shaarlier don't work with REST API enable in Shaarli (Tools/Configure). ppr |
Thank you for your feedback. So at least one issue is fixed :) For the other one, I have no clue 😅
What do you mean ? |
The copy/paste of an URL in the Shaarli web panel by clic on the "+Shaare" button is working. |
Describe the bug
I can't share an article/link through the Shaarlier app for Android. No error is returned but after about 10 tries I am banned by fail2ban. I use the secret code of the API to configure the sharing with Shaarlier. Sharing an article is normally done by logging in on the Shaarli web page without using the Shaarlier application.
Context
Steps to reproduce
Expected behavior
Provided that the Shaarlier application is set up correctly, it should normally be possible to share an article/link and not be banned by fail2ban.
The text was updated successfully, but these errors were encountered: