Add-on Firefox bootstrap.js

Version imprimable

Voir 40 message(s) de cette discussion en une page

Bonjour a vous tous

J'ai un add-on pour firefox dont la fonction est de bloquer une liste d'urls, malheureusement, ma liste d'urls se trouve a l'interieur de mon fichier bootstrap.js:

voici le script:

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
const {classes: Cc, interfaces: Ci, utils: Cu, results: Cr} = Components;
Cu.import('resource://gre/modules/Services.jsm');
 
var urls_block = [ //if urls contain any of these elements they will be blocked or redirected, your choice based on code in observer line 17
'URL1',
'URL2',
'URL3',
'etc...'];
 
var observers = {
    'http-on-modify-request': {
        observe: function (aSubject, aTopic, aData) {
            console.info('http-on-modify-request: aSubject = ' + aSubject + ' | aTopic = ' + aTopic + ' | aData = ' + aData);
            var httpChannel = aSubject.QueryInterface(Ci.nsIHttpChannel);
            var requestUrl = httpChannel.URI.spec;
            for (var i=0; i<urls_block.length; i++) {
             if (requestUrl.indexOf(urls_block[i]) > -1) {
              //httpChannel.cancel(Cr.NS_BINDING_ABORTED); //this aborts the load
              httpChannel.redirectTo(Services.io.newURI('data:text,CONTENT BLOCKED"', null, null)); //can redirect with this line, if dont want to redirect and just block, then uncomment this line and comment out line above (line 17)
              break;
}
            }
        },
        reg: function () {
            Services.obs.addObserver(observers['http-on-modify-request'], 'http-on-modify-request', false);
        },
        unreg: function () {
            Services.obs.removeObserver(observers['http-on-modify-request'], 'http-on-modify-request');
        }
    }
};
 
function install() {}
 
function uninstall() {}
 
function startup() {
 for (var o in observers) {
 	observers[o].reg();
 }
}
 
function shutdown(aData, aReason) {
 if (aReason == APP_SHUTDOWN) return;
 
 for (var o in observers) {
 	observers[o].unreg();
 }
}

le script fonctionne nickel, mais développeurs de Firefox me demandent de faire en sorte que la liste soit externe au bootstrap.js et q'elle soit loadee quand c'est necessaire.

merci a tous

10/12/2014, 23h02
beekeep

Bonjour,

il est possible de mettre les URLs dans un fichier texte et de les lire lors du premier chargement.

https://developer.mozilla.org/en-US/...o_text-streams
11/12/2014, 06h16
imothep85

Merci pour cette premiere reponse, comme je l'ai dit je ne suis pas expert en javascript.

Commment adapter donc ce code a mon add-on afin que ma database d'urls soit externe au bootstrap?

merci encore :D
12/12/2014, 12h04
lysandro

Bonjour,
Je ne connais rien à l'écriture des addons pour ffx, mais as tu regardé si tu pouvais utiliser une requêtre XMLHttpRequest ?
L'idée est que ton addons doit-être qque part sur le disque, peut-être peux tu fournir un fichier contenant les urls avec le addon et essayer de le charger avec une XHR.
Ou aller voir du coté de pref.js ?
12/12/2014, 18h36
imothep85

justement c'est ce qu'il faut il faut que le script charge les urls quand c'est necessaire sans quils soient en local, maintenant moi avec mon add-on, le script que je viens de montrer ici, je ne sais pas comment faire ca, je ne sais pas comment adapter mon code, pour loader les urls depuis un fichier externe
15/12/2014, 16h15
imothep85

mon problemme n'est toujours pas résolu :s
15/12/2014, 17h14
lysandro

C'est un problème d'écriture de add-on pour fxx, pas spécialement du javascript.
A ta place j'irais voir par là, mais sans garantie :

Une base :
https://developer.mozilla.org/en-US/...ped_extensions

Là, il y a des liens peut-être à consulter :
https://developer.mozilla.org/en-US/...urther_reading

C'est vieux mais ça en parle :
http://forums.mozillazine.org/viewto...f=19&t=2794811

dans la fonction startup, tu récupères une structure de données avec :
installPath:nsIFile et resourceURI:nsIURI, doit y avoir moyen de l'utiliser

peut-être voir du coté nsIChannel, ou https://developer.mozilla.org/en-US/...XMLHttpRequest

Et le lien que t'as donné beekeep l'as tu exploité ?
15/12/2014, 22h54
SylvainPV

La fonction readTextFromFile dans le lien de beekeep semble être exactement ce que tu cherches :

Code:

var urls_block =JSON.parse(readTextFromFile("taliste.json"));
16/12/2014, 00h43
imothep85

Salut SylvainPv, comme tu le sais juis bloqué, je lis pas mal d'articles mais mon bleme c'est comment, et ou modifier mon add-on pour que ce nouveau systeme finctionne
16/12/2014, 00h56
SylvainPV

Et que fais-tu des trois précédentes réponses qu'on t'a donné ? On t'a dit tout ce qu'il y a à savoir... :roll:
16/12/2014, 01h01
imothep85

et bien si je change mon code par

Code:

var urls_block =JSON.parse(readTextFromFile("taliste.json"));

le chemin ou se trouve le fichier contenant les urls reste visible :s c'est dangereux non? et de plus est ce que cela fonctionnera juste avec cette modif?
16/12/2014, 02h49
SylvainPV

D'une manière ou d'une autre il faudra bien que le client récupère la liste donc je ne vois pas ce que ça change question sécurité. Par rapport à la sécurité, on avait déjà évoqué le sujet dans ton premier topic sur le même projet et tu avais écarté l'option de faire le test d'URL côté serveur. Donc pas le choix, il faut la charger côté client, mais de ce que tu nous as dit sur les remarques de la team Mozilla, un fichier externe semble leur convenir.

Citation:

de plus est ce que cela fonctionnera juste avec cette modif?

Et si tu essayais pour voir ?
16/12/2014, 08h26
Bovino

Citation:

le fichier contenant les urls reste visible :s c'est dangereux non?

Le fichier se trouve sur le poste du client, en quoi ça pourrait être dangereux que l'utilisateur de l'add-on puisse savoir qu'il y a un autre fichier sur son poste ? :koi:
03/01/2015, 22h27
imothep85

et bien si le script contenant tous les urls est visible pour l'utilisateur final c'est pas rassurant ca peut etre mal utilisé.
C'est l'equippe de mozilla qui ma demandé de faire en sorte que les urls soitn dans un fichier externe a uscript bootstrap.js, et que ces urls soient appelés, quand il le faut seulement.
Seulement voila, je suis toujours bloqué avec ce problemme.
nous sommes maintenant en debut 2015 et j'aimerais vraiment finaliser les script :s
j'ai plus de 6000 utilisateurs de mon petit add-on, et on ma deja fait la remarque a propos des urls visibles pour l'utilisateur :s .
04/01/2015, 16h39
SylvainPV

Et le problème n'a pas bougé depuis 2014, tu te confrontes toujours au même choix: soit faire le boulot côté client ce qui implique que la liste d'URL soit accessible à tous les utilisateurs, soit faire un service web dédié à interroger à chaque requête de chacun de tes clients, ce qui est très coûteux à la fois pour toi et pour tes utilisateurs. Note que la solution serveur revient plus ou moins à faire un serveur proxy, donc l'extension Firefox ne servirait plus à rien en elle-même.
04/01/2015, 18h02
lysandro

Même si tes urls sont dans ton bootstrap.js, les extensions sont installées au mieux comme .jar, c'est à dire que le contenu (code et données) est accessible à qui veut bien se donner la peine de le chercher. Autrement dit, à l'intérieur de bootstrap.js ou en fichier à coté en terme de sécurité c'est la même chose. Tu peux toujours distribuer ton fichier d'urls dans le fichier .jar de l'application, il sera moins visible, si ça te rassure.
04/01/2015, 20h35
imothep85

Lysandro oui ce serait une idée a exploiter,
le tout pour moi est de faire en sorte que la liste d'urls soti externe au bootstrap.js
mais je ne sais pas comment faire ces changements dans mon add-on, cest cela que j'essaye de voir fini, le bootstrap.js appele le fichier qui sera dans le jar contenant tous les urls.
06/01/2015, 23h41
imothep85

quelqun peut t'il m'aider a finaliser mon problemme?
06/01/2015, 23h47
beekeep

Est-ce qu'on pourrait voir la réponse des éditeurs lors de la revue de ton module ?
07/01/2015, 00h13
imothep85

voila leur reponse:

As mentioned in Version 1.4.30 · October 10, 2014 review, v1.4.45 review & v1.46 review:
Please place database code in a separate file and load it in bootstrap.js when necessary.

Please fix them and submit again. Thank you.
07/01/2015, 00h20
beekeep

Est-ce que tu as mis les URLs dans un fichier à part et utilisé le code indiqué dans mon premier lien pour les charger ?
07/01/2015, 00h23
imothep85

et bien je my perds vraiment, par rapport a mon code je ne sais pas ce que je dois modifier, poru que cela marche, et si je sauve mes urls dans un fichier externe en txt ou js.
cets pour cela en fait que je demande si c'est possible d'avoir le code de mon add-on deja modifié afin que cela fonctionne :D comme je l'ai dit je ne suis malheureusement ps un genie comme certains ici avec javascript
07/01/2015, 00h34
beekeep

Je ne suis pas sur que quelqu'un veuille le faire à ta place.

Tu auras plus d'aide en essayant de comprendre et de demander de l'aide sur un problème en particulier.

Tu peux utiliser un fichier texte.
Ensuite il faut lire le fichier et remplir un tableau de chaine de caractères avec les URLs.
07/01/2015, 00h43
imothep85

lire le fichier texte et et remplir un tableau de chaine de caractères avec les URLs??
et ceci

Code:

var urls_block =JSON.parse(readTextFromFile("taliste.json"));

si je remplace ("taliste.json")) par ("blocklist.json")) ? et que dans ce fichier json je mets tous mes urls?
07/01/2015, 10h04
beekeep

Oui c'est un bon début.
07/01/2015, 16h17
imothep85

donc j'ai procédé comem je l'avais dit precedament mais evidament ca ne fonctionne pas.
Dans mon fichier json j'ai mis tous mes urls, mais que dosi je mettre en debut et fin de fichier pour que ca marche???
mettre juste les urls ca ne fonctionne pas!!
08/01/2015, 11h38
beekeep

Si tu choisis le format texte tu peux mettre une URL par ligne.
Avec un format JSON il faut mettre un objet avec un tableau contenant les URLs au format JSON.
08/01/2015, 11h47
imothep85

Donc si ma liste d'urls est un fichier texte au lieu d'un fichier json ca devrait aller??
si dans mon bootstratp.js je mets.

Code:

var urls_block =JSON.parse(readTextFromFile("taliste.txt"));
08/01/2015, 14h40
beekeep

Pas besoin de JSON.parse si ce n'est pas un fichier JSON.
08/01/2015, 15h39
imothep85

et donc j'ai

Code:

var urls_block =JSON.parse(readTextFromFile("taliste.txt"));

je le remplace par

Code:

var urls_block =???(readTextFromFile("taliste.txt"));

quoi donc?
08/01/2015, 16h10
beekeep

La fonction readTextFromFile renvoi tout le texte du fichier, il faut ensuite utiliser un Split pour diviser la chaine en plusieurs en utilisant le séparateur saut de ligne (\n).
08/01/2015, 16h22
imothep85

Merci encore poru ton aide :D
donc ca je fais quoi exactement?

Code:

var urls_block =???(readTextFromFile("taliste.txt"));

et dans mon fichier liste.txt j'ai que des urls separés par des ;

que faut t'il que je mette avant les urls et en fin d'urls? rien?
08/01/2015, 17h25
beekeep

Si elles sont séparées par des points virgules:

Code:

var urls_block = readTextFromFile("taliste.txt").Split(';');

Rebonjour donc atuellement j'ai ceci

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
const {classes: Cc, interfaces: Ci, utils: Cu, results: Cr} = Components;
Cu.import('resource://gre/modules/Services.jsm');
 
var urls_block = readTextFromFile("url.txt").Split(',');
 
var observers = {
    'http-on-modify-request': {
        observe: function (aSubject, aTopic, aData) {
            console.info('http-on-modify-request: aSubject = ' + aSubject + ' | aTopic = ' + aTopic + ' | aData = ' + aData);
            var httpChannel = aSubject.QueryInterface(Ci.nsIHttpChannel);
            var requestUrl = httpChannel.URI.spec;
            for (var i=0; i<urls_block.length; i++) {
             if (requestUrl.indexOf(urls_block[i]) > -1) {
              //httpChannel.cancel(Cr.NS_BINDING_ABORTED); //this aborts the load
              httpChannel.redirectTo(Services.io.newURI('data:text,ILLEGAL & DANGEROUS CONTENT BLOCKED"', null, null)); //can redirect with this line, if dont want to redirect and just block, then uncomment this line and comment out line above (line 17)
              break;
}
            }
        },
        reg: function () {
            Services.obs.addObserver(observers['http-on-modify-request'], 'http-on-modify-request', false);
        },
        unreg: function () {
            Services.obs.removeObserver(observers['http-on-modify-request'], 'http-on-modify-request');
        }
    }
};
 
function install() {}
 
function uninstall() {}
 
function startup() {
 for (var o in observers) {
 	observers[o].reg();
 }
}
 
function shutdown(aData, aReason) {
 if (aReason == APP_SHUTDOWN) return;
 
 for (var o in observers) {
 	observers[o].unreg();
 }
}

dans mon fichier externe url.txt
mes urls sont comem ceci:
'url01',
'url03',
etc...
mais mon addon ne bloque plus ces urls

08/01/2015, 22h42
beekeep
Tu peux déboguer le code en utilisant la "boite à outils du navigateur" disponible via le menu outils, développent web.
(il y a un champ pour rechercher un script par son nom)

De cette façon tu peux vérifier que le chargement se passe comme il faut (en modifiant ce qui ne va pas).

Le chargement des URLs doit être fait uniquement au premier test et non au chargement du module pour ne pas ralentir Firefox.

Initialiser la variable à null:

Code:

var urls_block = null;

puis au début de la fonction observe charger les URLs si besoin:
Code:

1 2 3 4 if(!urls_block) { urls_block = readTextFromFile("url.txt").Split(','); }

je lance le module ardoise javascript quand j'analyse mon script ca me dit en fin de fichier

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
const {classes: Cc, interfaces: Ci, utils: Cu, results: Cr} = Components;
Cu.import('resource://gre/modules/Services.jsm');
 
var urls_block = null;
 
if(!urls_block)
{
  urls_block = readTextFromFile("url.txt").Split(',');
}
var observers = {
    'http-on-modify-request': {
        observe: function (aSubject, aTopic, aData) {
            console.info('http-on-modify-request: aSubject = ' + aSubject + ' | aTopic = ' + aTopic + ' | aData = ' + aData);
            var httpChannel = aSubject.QueryInterface(Ci.nsIHttpChannel);
            var requestUrl = httpChannel.URI.spec;
            for (var i=0; i<urls_block.length; i++) {
             if (requestUrl.indexOf(urls_block[i]) > -1) {
              //httpChannel.cancel(Cr.NS_BINDING_ABORTED); //this aborts the load
              httpChannel.redirectTo(Services.io.newURI('data:text,ILLEGAL & DANGEROUS CONTENT BLOCKED"', null, null)); //can redirect with this line, if dont want to redirect and just block, then uncomment this line and comment out line above (line 17)
              break;
}
            }
        },
        reg: function () {
            Services.obs.addObserver(observers['http-on-modify-request'], 'http-on-modify-request', false);
        },
        unreg: function () {
            Services.obs.removeObserver(observers['http-on-modify-request'], 'http-on-modify-request');
        }
    }
};
 
function install() {}
 
function uninstall() {}
 
function startup() {
 for (var o in observers) {
 	observers[o].reg();
 }
}
 
function shutdown(aData, aReason) {
 if (aReason == APP_SHUTDOWN) return;
 
 for (var o in observers) {
 	observers[o].unreg();
 }
}

/*
Exception: redeclaration of const Cc
@Scratchpad/1:1:17
WCA_evalWithDebugger@resource://gre/...lkit/loader.js -> resource://gre/modules/devtools/server/actors/webconsole.js:1122:16
WCA_onEvaluateJS@resource://gre/modu...lkit/loader.js -> resource://gre/modules/devtools/server/actors/webconsole.js:750:20
DSC_onPacket@resource://gre/modules/...lkit/loader.js -> resource://gre/modules/devtools/server/main.js:1233:15
LocalDebuggerTransport.prototype.send/<@resource://gre/modules/devtools/dbg-client.jsm -> resource://gre/modules/devtools/transport/transport.js:545:11
makeInfallible/<@resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/devtools/DevToolsUtils.js:83:14
makeInfallible/<@resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/devtools/DevToolsUtils.js:83:14

09/01/2015, 12h47
lysandro

Histoire de faire avancer le schmilblick, j'ai testé :

readTextFromFile - Je pense qu'il faut copier le source de la fonction dans le code, elle n'était pas définie dans mon environnement. Ensuite, je n'ai jamais pu trouver comment intégrer 'require' qui semble aussi non définie. Je pense, mais je ne l'ai pas validé, qu'il faut installer le sdk pour que ça fonctionne et générer le add-on à partir du sdk. En passant, '.split' s'écrit avec un 's' minuscule.

Sinon il y a une alternative https://developer.mozilla.org/en-US/...XMLHttpRequest

edit:10/01/2015-1050 - retrait code inutile
09/01/2015, 13h59
beekeep

Citation:

Envoyé par imothep85

je lance le module ardoise javascript quand j'analyse mon script ca me dit en fin de fichier

Code:

const {classes: Cc, interfaces: Ci, utils: Cu, results: Cr} = Components;

Citation:

Exception: redeclaration of const Cc

Il faut donc changer le nom des constantes que tu utilises. (Cc, Ci, Cu, Cr)

:fleche: Essayes de lire les messages d'erreur, de les comprendre et d'essayer de corriger les erreurs toi-mêmes.
09/01/2015, 14h02
beekeep

Citation:

Envoyé par lysandro

Histoire de faire avancer le schmilblick

Franchement je ne pense pas que ça fasse trop avancer les choses en changeant complètement le code. (Charger un fichier local avec une XHR ??)

Il faudrait tester dans le bon environnement (add-on via jetpack).
09/01/2015, 14h36
lysandro

C'est pas faux

edit:10/01/2015-10:50

Voir 40 message(s) de cette discussion en une page