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

Jump to bottom

Cannot save setting "Also use the Anonymized IP addresses when enriching visits." #19089

Closed
kaskaes opened this issue Apr 13, 2022 · 2 comments · Fixed by #19094
Closed

Cannot save setting "Also use the Anonymized IP addresses when enriching visits." #19089

kaskaes opened this issue Apr 13, 2022 · 2 comments · Fixed by #19094
Labels
Bug For errors / faults / flaws / inconsistencies etc. not-in-changelog For issues or pull requests that should not be included in our release changelog on matomo.org. Regression Indicates a feature used to work in a certain way but it no longer does even though it should.
Milestone
4.9.1

Comments

@kaskaes
Copy link

kaskaes commented Apr 13, 2022 •
edited

Expected Behavior

Under Administration -> Privacy -> Anonymize data -> "Also use the Anonymized IP addresses when enriching visits." the radio button ist set to "Yes". If I change it to "No" and press "save" Button, the value should be stored.

Current Behavior

Under Administration -> Privacy -> Anonymize data -> "Also use the Anonymized IP addresses when enriching visits." the radio button ist set to "Yes". If I change it to "No" and press "save" Button, it looks like the new value is stored. But after reloading / refreshing the page, the value (radio button) is still "Yes"

Possible Solution

Steps to Reproduce (for Bugs)

  1. Go to Administration Administration -> Privacy -> Anonymize data
  2. Change the radio button under "Also use the Anonymized IP addresses when enriching visits." to "No"3.
  3. Press "Save" Button
  4. Reload the Page
  5. Value has not changed

#Context

Your Environment

  • Matomo Version: 4.9.0
  • PHP Version: 8.1.3
  • Server Operating System: Linux
  • Additionally installed plugins:
  • Browser: Safari 15.4
  • Operating System: Mac OS 12.3.1
@kaskaes kaskaes added the Potential Bug Something that might be a bug, but needs validation and confirmation it can be reproduced. label Apr 13, 2022
@sgiehl sgiehl added Bug For errors / faults / flaws / inconsistencies etc. and removed Potential Bug Something that might be a bug, but needs validation and confirmation it can be reproduced. labels Apr 13, 2022
@sgiehl sgiehl added this to the 4.9.1 milestone Apr 13, 2022
@sgiehl sgiehl added the Regression Indicates a feature used to work in a certain way but it no longer does even though it should. label Apr 13, 2022
@sgiehl
Copy link
Member

sgiehl commented Apr 13, 2022

Hi @kaskaes. Thanks for your report. I was able to reproduce that.

@diosmosis that seems to be vue related. The request always contains useAnonymizedIpForVisitEnrichment: 1, even if the no option was selected. Could you have a quick look?

@diosmosis diosmosis mentioned this issue Apr 13, 2022
Merged
11 tasks
@sgiehl sgiehl closed this as completed Apr 14, 2022
@MatomoForumNotifications
Copy link

This issue has been mentioned on Matomo forums. There might be relevant details there:

https://forum.matomo.org/t/anonymize-tracking-data-issue-after-update-to-4-9/45565/4

@justinvelluppillai justinvelluppillai added the not-in-changelog For issues or pull requests that should not be included in our release changelog on matomo.org. label May 8, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Bug For errors / faults / flaws / inconsistencies etc. not-in-changelog For issues or pull requests that should not be included in our release changelog on matomo.org. Regression Indicates a feature used to work in a certain way but it no longer does even though it should.
Projects
None yet
Milestone
4.9.1
Development

Successfully merging a pull request may close this issue.

correctly handle string 0/1 values for useAnonymizedIpForVisitEnrichment matomo-org/matomo
4 participants
@justinvelluppillai @sgiehl @MatomoForumNotifications @kaskaes