Signature: File is missing. Should it be signed with the deb.torproject.org archive signing key (8B904624C5A28654E4539BC2E135A8B41A7BF184)?
I don't understand what the Debian archive has to do with the security policy.
I suggest we use the tor-security list key, or some other key that many people trust.
Also, how did this patch get merged without a signature?
Please open a ticket to remove the signature line, because it looks broken.
Then, please open a ticket to get it signed.
(Please don't change the titles of tickets to a different task, that's confusing. And integers are cheap.)
Hiring: it could help the Torproject to always have an open position for security researchers
Tor security researchers typically work for universities or similar organisations, or are freelance, or are volunteers.
Since we don't have open security-related jobs, please open a ticket to remove the hiring line.
Also, Tor doesn't always have positions open in any category.
Changing this part of Tor is best done by talking to HR, the executive director, or the affected teams. It really isn't a good topic for a trac ticket.
Who's willing to adopt this ticket?
If you open separate tickets for each task, different people might adopt those tasks.
Also, how did this patch get merged without a signature?
I guess it was merged without reading the PR description or the according ticket.
If you open separate tickets for each task, different people might adopt those tasks.
Done.
I am confused by the commits in this pull request. It looks like some of them have already been merged to master?
It looks like you tried to add a commit called "rebase".
Instead, please rebase the branch onto master, and open a new pull request with the result.
Here are my comments on the original commit to the contact page:
I don't understand what the "acknowledgements" are.The network team security policy is *not* the security policy for the whole of Tor.The link to .well_known/security.txt is blank. Does that work?
(learned something: I used the already merged branch securitytxt, added the commit and did 'git rebase master'. So all commits got a new hash. Instead I need to deleted the merged branch, checkout -b securitytxt and add the commit there.)
The network team security policy is not the security policy for the whole of Tor.
Yes, the current link does not make much sense. I propose to make up a (wiki?) path like TorSecurityPolicy to be filled later. It probably should also appear on WikiStart#OfficialTorLinks.
The link to .well_known/security.txt is blank. Does that work?
Got me. The whole PR was WIP and I did not expect it to be merged at this stage.
The network team security policy is not the security policy for the whole of Tor.
Yes, the current link does not make much sense. I propose to make up a (wiki?) path like TorSecurityPolicy to be filled later. It probably should also appear on WikiStart#OfficialTorLinks.
Ok, but you can't link to a blank policy. That's confusing.
I suggest that you point to the network team security policy as the policy for "tor (the network daemon)", and note that other projects may have their own policies.
If you link to a wiki, then we can add new policies easily, but the content is hard to trust.
If you get the security.txt file signed, then it's harder to update the links.
Please check the links, they have typos.
And please be consistent with your onion links: either provide onion links for all links, or no links.