This repository has been archived by the owner on Feb 6, 2023. It is now read-only.
Redirect-rule modifier description #817
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
vbagirov
suggested changes
May 11, 2021
| @@ -980,6 +981,19 @@ This rule redirects all requests to `example.org/test.mp4` to the resource named | |||
|
|
|||
| > **Compatibility with different versions of AdGuard.** This type of rules is not supported by AdGuard for iOS and Safari. | |||
|
|
|||
| <a id="redirect-rule-modifier"></a> | |||
| #### `redirect-rule` | |||
There was a problem hiding this comment.
В жирненький надо завернуть, чтобы было как везде
| @@ -980,6 +981,19 @@ This rule redirects all requests to `example.org/test.mp4` to the resource named | |||
|
|
|||
| > **Compatibility with different versions of AdGuard.** This type of rules is not supported by AdGuard for iOS and Safari. | |||
|
|
|||
| <a id="redirect-rule-modifier"></a> | |||
| #### `redirect-rule` | |||
| This is basically an alias to $redirect since it has the same "redirection" values and the logic is almost similar. The difference is that $redirect-rule is applied only in the case when the target request is blocked by a different basic rule. | |||
There was a problem hiding this comment.
$redirect и $redirect-rule завернуть в такие кавычки: `
|
|
||
| Examples: | ||
|
|
||
| `||example.org/script.js` |
There was a problem hiding this comment.
Строчки с примерами завернуть сверху и снизу в тройные кавычки: ```
vbagirov
previously approved these changes
May 11, 2021
sxgunchenko
suggested changes
May 11, 2021
|
@sxgunchenko смысл был проверить суть на англ, если там всё ок, то переводить на ру. |
vbagirov
suggested changes
May 11, 2021
| #### **`redirect-rule`** | ||
| По сути, это тот же `$redirect`, поскольку у него тоже есть свойства "перенаправления" и логика у них практически аналогичная. Разница в том, что `$redirect-rule` применяется только в случае, если целевой запрос заблокирован другим базовым правилом. | ||
|
|
||
| > Отрицание `$redirect-rule` работает точно так же, как для обычных правил `$redirect`. Даже более того, `@@|||example.org^$redirect` будет отрицать как правило `$redirect`, так и `$redirect-rule`. |
There was a problem hiding this comment.
Давай тут везде "отрицание" заменим на "отмену"
sxgunchenko
suggested changes
May 11, 2021
| #### **`redirect-rule`** | ||
| По сути, это тот же `$redirect`, поскольку у него тоже есть свойства "перенаправления" и логика у них практически аналогичная. Разница в том, что `$redirect-rule` применяется только в случае, если целевой запрос заблокирован другим базовым правилом. | ||
|
|
||
| > Отрицание `$redirect-rule` работает точно так же, как для обычных правил `$redirect`. Даже более того, `@@|||example.org^$redirect` будет отрицать как правило `$redirect`, так и `$redirect-rule`. |
There was a problem hiding this comment.
тут скорее не отрицание, а отключение, как в других модификаторах (например, $replace)
| #### **`redirect-rule`** | ||
| По сути, это тот же `$redirect`, поскольку у него тоже есть свойства "перенаправления" и логика у них практически аналогичная. Разница в том, что `$redirect-rule` применяется только в случае, если целевой запрос заблокирован другим базовым правилом. | ||
|
|
||
| > Отрицание `$redirect-rule` работает точно так же, как для обычных правил `$redirect`. Даже более того, `@@|||example.org^$redirect` будет отрицать как правило `$redirect`, так и `$redirect-rule`. |
There was a problem hiding this comment.
лишняя палка в примере правила: @@|||example...
slavaleleka
suggested changes
May 11, 2021
| #### **`redirect-rule`** | ||
| По сути, это тот же `$redirect`, поскольку у него тоже есть свойства "перенаправления" и логика у них практически аналогичная. Разница в том, что `$redirect-rule` применяется только в случае, если целевой запрос заблокирован другим базовым правилом. | ||
|
|
||
| > Отрицание `$redirect-rule` работает точно так же, как для обычных правил `$redirect`. Даже более того, `@@|||example.org^$redirect` будет отрицать как правило `$redirect`, так и `$redirect-rule`. |
There was a problem hiding this comment.
@@||example.org^$redirect — то есть два символа |
There was a problem hiding this comment.
первое поправила, со вторым не согласна
| @@ -975,6 +976,21 @@ AdGuard использует тот же синтаксис правил фил | |||
|
|
|||
| > **Совместимость с различными версиями AdGuard.** Этот тип правил не поддерживается AdGuard для iOS и для Safari. | |||
|
|
|||
| <a id="redirect-rule-modifier"></a> | |||
| #### **`redirect-rule`** | |||
| По сути, это тот же `$redirect`, поскольку у него тоже есть свойства "перенаправления" и логика у них практически аналогичная. Разница в том, что `$redirect-rule` применяется только в случае, если целевой запрос заблокирован другим базовым правилом. | |||
There was a problem hiding this comment.
"перенаправления" — кавычки не кириллические
There was a problem hiding this comment.
у нас в этой статье (и в многих других), увы, используются эти
sxgunchenko
reviewed
May 11, 2021
| #### **`redirect-rule`** | ||
| По сути, это тот же `$redirect`, поскольку у него тоже есть свойства "перенаправления" и логика у них практически аналогичная. Разница в том, что `$redirect-rule` применяется только в случае, если целевой запрос заблокирован другим базовым правилом. | ||
|
|
||
| > Отмена `$redirect-rule` работает точно так же, как для обычных правил `$redirect`. Даже более того, `@@|||example.org^$redirect` будет отрицать как правило `$redirect`, так и `$redirect-rule`. |
There was a problem hiding this comment.
осталось отрицать: "...будет отрицать как..."
slavaleleka
suggested changes
May 11, 2021
sxgunchenko
approved these changes
May 11, 2021
vbagirov
approved these changes
May 12, 2021
slavaleleka
approved these changes
May 12, 2021
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Closes #815