search

stape-io / mailchimp-tag

Mailchimp Tag for Google Tag Manager Server Side
https://stape.io/connect-your-website-to-mailchimp-using-google-tag-manager-server-container/
Apache License 2.0
2 stars 0 forks source link

Use encodeUri() if part of the URL path #7

Closed opozo closed 6 months ago

opozo commented 6 months ago

When reviewing your template we've noticed that it is using user-defined parameters to build a URL. It's best practice to use encodeUri() if part of the URL path [1] [2].

Bukashk0zzz commented 6 months ago

Thanks for the issue. This fixed in https://github.com/stape-io/mailchimp-tag/commit/26e19327dc73a3f1b692dab31d02a678e19b4b1c