Add comments about auth
This commit is contained in:
parent
34091038f3
commit
708f281270
1 changed files with 4 additions and 0 deletions
|
@ -123,6 +123,8 @@ func serveSpec(w http.ResponseWriter, req *http.Request) {
|
||||||
return
|
return
|
||||||
}
|
}
|
||||||
|
|
||||||
|
// We're going to run whatever Python is specified in the pull request, which
|
||||||
|
// may do bad things, so only trust people we trust.
|
||||||
if !allowedMembers[pr.User.Login] {
|
if !allowedMembers[pr.User.Login] {
|
||||||
w.WriteHeader(403)
|
w.WriteHeader(403)
|
||||||
io.WriteString(w, fmt.Sprintf("%q is not a trusted pull requester", pr.User.Login))
|
io.WriteString(w, fmt.Sprintf("%q is not a trusted pull requester", pr.User.Login))
|
||||||
|
@ -158,6 +160,8 @@ func serveRstDiff(w http.ResponseWriter, req *http.Request) {
|
||||||
return
|
return
|
||||||
}
|
}
|
||||||
|
|
||||||
|
// We're going to run whatever Python is specified in the pull request, which
|
||||||
|
// may do bad things, so only trust people we trust.
|
||||||
if !allowedMembers[pr.User.Login] {
|
if !allowedMembers[pr.User.Login] {
|
||||||
w.WriteHeader(403)
|
w.WriteHeader(403)
|
||||||
io.WriteString(w, fmt.Sprintf("%q is not a trusted pull requester", pr.User.Login))
|
io.WriteString(w, fmt.Sprintf("%q is not a trusted pull requester", pr.User.Login))
|
||||||
|
|
Loading…
Add table
Add a link
Reference in a new issue