matomo-org
Update base.rb,fixed matomo API call
Description:
Fixed gem work with new version matomo, changed the API call from get to post
Thank you for the pull request.
I think, loosening the ActiveSupport constraint is the right call.
But why did you change the request method from GET to POST? Which problem does this solve?
Thank you for the pull request.
I think, loosening the ActiveSupport constraint is the right call.
But why did you change the request method from GET to POST? Which problem does this solve?
Hey!
I was just checking out the gem, and found this PR.
Based on matomo documentation, when using an auth_token which has the Secure use only feature enabled, you won't be able to use it via a GET request. I'm assuming that's the reason behind this change.
Sounds plausible.
If that is the case, personally I'd prefer to keep using GET and only use POST when an auth_token is provided. At least until I get some more feedback on that this doesn't break anything for other users and/or use cases.
Anyway, I no longer have merge authorization, so it doesn't actually matter...