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

Tags and lists are public by default despite private default setting #644

Closed
chrissawyerfan4 opened this issue May 3, 2023 · 2 comments · Fixed by #652
Closed

Tags and lists are public by default despite private default setting #644

chrissawyerfan4 opened this issue May 3, 2023 · 2 comments · Fixed by #652
Labels
Bug Something isn't working

Comments

@chrissawyerfan4
Copy link
Contributor

chrissawyerfan4 commented May 3, 2023

Bug Description

Newly created tags and lists can be viewed by guests despite these settings:

$ sqlite3 storage/database.sqlite
sqlite> select * from settings where key like '%private_default';
id|user_id|key|value
1|1|links_private_default|1
13|1|notes_private_default|1
14|1|lists_private_default|1
15|1|tags_private_default|1

How to reproduce

  1. Log in on https://demo.linkace.org
  2. Configure the system to have private lists or tags by default
  3. Click the button to create a new link
  4. Add a list or tag for this link which does not yet exist in the system
  5. Open https://demo.linkace.org/guest/tags or https://demo.linkace.org/guest/lists

Expected behavior

It respects the setting, or assumes a private default which is less likely to lead to an issue for the user

Presumably this can be fixed here https://github.com/Kovah/LinkAce/blob/main/app/Repositories/TagRepository.php#L20 but I don't know this framework that well to know what function to call to check this setting, and was hoping this might be a 3-minute change for you

Client details

Verified on demo system running v1.12.0, at least for tags (I assume lists is the same, as I had this issue locally as well)

@chrissawyerfan4 chrissawyerfan4 added the Bug Something isn't working label May 3, 2023
@Kovah
Copy link
Owner

Kovah commented May 4, 2023

Hi, thanks for taking your time to report this and even verifying it on the demo setup. Will have a look as soon as possible.

@Kovah Kovah linked a pull request Jun 4, 2023 that will close this issue
@Kovah Kovah closed this as completed in #652 Jun 4, 2023
@chrissawyerfan4
Copy link
Contributor Author

Thank you for resolving this! :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Something isn't working
Projects
Status: Released
Development

Successfully merging a pull request may close this issue.

2 participants