We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
AdGuardHome/client/src/actions/index.js
Line 624 in 69f526b
According to the documentation, it says
If the client name contains quotes, use \ to escape them.
https://github.com/AdguardTeam/AdGuardHome/wiki/Hosts-Blocklists#client
So the code should be
const baseRule = `||${domain}^$client='${client.replace(/'/g, '\\\'')}'`;
The text was updated successfully, but these errors were encountered:
Oh, thanks for noticing this!
Sorry, something went wrong.
42b2c3a
Pull request: 3201 use backslash to escape quotes in the client name
5e83c8f
Closes AdguardTeam#3201 Squashed commit of the following: commit 43d2b96 Merge: 36eeb1b 784bc31 Author: Ildar Kamalov <ik@adguard.com> Date: Mon Aug 16 17:29:06 2021 +0300 Merge branch 'master' into 3201-escape-quotes commit 36eeb1b Author: Ildar Kamalov <ik@adguard.com> Date: Mon Aug 16 15:56:15 2021 +0300 client: multiple escape commit e7c19dd Author: Ildar Kamalov <ik@adguard.com> Date: Mon Aug 16 12:04:43 2021 +0300 client: use backslash to escape quotes in the client name
IldarKamalov
No branches or pull requests
AdGuardHome/client/src/actions/index.js
Line 624 in 69f526b
According to the documentation, it says
https://github.com/AdguardTeam/AdGuardHome/wiki/Hosts-Blocklists#client
So the code should be
The text was updated successfully, but these errors were encountered: