AdguardTeam / AdguardTeam/Scriptlets

New approach to scriptlets

Abierto
#83 0 comentarios 0 reacciones 1 asignado Reclamado por @ameshkov Ver en GitHub
enhancement Priority: P4 Version: Scriptlets v3.0
Lenguaje dominante
JavaScript
Estrellas
195
Forks
33
Métricas de merge de PR
Sin PR fusionados en 30 d

Descripción

_Originally posted by @ameshkov in https://github.com/AdguardTeam/Scriptlets/issues/82#issuecomment-625230500_

Check out all the comments in the thread.

Here are some thoughts on this.

Currently, there are two important scriptlets groups:

* Override a property get/set and do something.
* Override a function call and do something.

The algorithm, in general, may look like this:

1. Override a property or a function.
2. Once it's called, check the "matchers" specified in the rule: stack trace, script content, function call arguments.
3. Apply one of the possible actions.

Here are some of the existing scriptlets written this way (don't mind the naming, I am coming up with this right now):
```
! noeval
function('window.eval'):match-argument(1, /regex/):noop()
! set-constant
property('window.test', 'get'):return-constant(true)
! log-eval
function('window.eval):log()'
! prevent-setInterval / no-setInterval-if
function('window.setInterval'):match-argument(1, /regex/):match-argument(2, 500):noop()
! json-prune
function('JSON.parse'):match-argument-json-props(1, 'someProp'):remove-json-props-and-return(1, 'someProp')
```
> I am not sure about the `JSON.parse` example, though. Having the capability to apply this kind of modification to an arbitrary function call can be dangerous in the wrong hands.

This approach makes adding new matching criteria really easy. Just a couple of examples that can be useful: `match-url`, `match-selector`, etc.

Some scriptlets that cannot be categorized as "function", "property" or "matcher" will be categorized as "actions", and it should still be possible to apply some "matchers" to them. For instance:
```
! run remove-attr only on a specific page
:match-url('some-page-url'):remove-attr('onclick', 'selector>here')
```

Guía de contribución

No hay ninguna guía de contribución indexada para este repositorio

Evaluación

Este issue todavía no se ha evaluado.

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.