[Author Prev][Author Next][Thread Prev][Thread Next][Author Index][Thread Index]
[tor-bugs] #10815 [EFF-HTTPS Everywhere]: 4.dev.15 ruleset db is not a proper db?
#10815: 4.dev.15 ruleset db is not a proper db?
----------------------------------+---------------------
Reporter: Faziri | Owner: pde
Type: enhancement | Status: new
Priority: normal | Milestone:
Component: EFF-HTTPS Everywhere | Version:
Keywords: | Actual Points:
Parent ID: | Points:
----------------------------------+---------------------
First off, thank you for resolving
https://trac.torproject.org/projects/tor/ticket/10174
However, the implementation looks wrong. The database does not appear to
be made the way an SQL db is supposed to be made. There could be a reason
that'd warrant an exception, so I'm mostly just asking for a clarification
as to why this odd choice was made, not complaining that the db is bad.
http://puu.sh/6M0HH.png
The db contains a column "id" to link domains and a column "contents",
'''which simply contains the lines of XML'''. You've essentially replaced
the text file that contains lines of XML by a db that contains lines of
XML.
Normally speaking, the db should have a column for each of the properties
of a rule.
This allows for lazy loading of the ruleset, so the problem was solved on
the filesystem level, but it was not solved on the per-rule level since
it's still just XML that needs to be parsed. And now, you have a database
that violates database design rules. Using the db to the fullest by
transforming the XML rules into db columns would further improve the
system.
So why was the choice made to do it this way?
Cheers
--
Ticket URL: <https://trac.torproject.org/projects/tor/ticket/10815>
Tor Bug Tracker & Wiki <https://trac.torproject.org/>
The Tor Project: anonymity online
_______________________________________________
tor-bugs mailing list
tor-bugs@xxxxxxxxxxxxxxxxxxxx
https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-bugs