pub-solar/forgejo
5
0
Fork 0
Code Issues Pull requests Packages Projects Releases Wiki Activity

ignore DNS error when doing migration allow/block check (#19566)

Browse source 
Co-authored-by: 6543 <6543@obermui.de>
This commit is contained in:
wxiaoguang 2022-05-02 12:02:17 +08:00 committed by GitHub
parent 05234adfa8
commit 71bafa0263
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
2 changed files with 3 additions and 8 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
models/error.go
View file

@ -296,7 +296,6 @@ type ErrInvalidCloneAddr struct {
IsProtocolInvalid bool IsProtocolInvalid bool
IsPermissionDenied bool IsPermissionDenied bool
LocalPath bool LocalPath bool
NotResolvedIP bool
} }
// IsErrInvalidCloneAddr checks if an error is a ErrInvalidCloneAddr. // IsErrInvalidCloneAddr checks if an error is a ErrInvalidCloneAddr.
@ -306,9 +305,6 @@ func IsErrInvalidCloneAddr(err error) bool {
} }
func (err *ErrInvalidCloneAddr) Error() string { func (err *ErrInvalidCloneAddr) Error() string {
if err.NotResolvedIP {
return fmt.Sprintf("migration/cloning from '%s' is not allowed: unknown hostname", err.Host)
}
if err.IsInvalidPath { if err.IsInvalidPath {
return fmt.Sprintf("migration/cloning from '%s' is not allowed: the provided path is invalid", err.Host) return fmt.Sprintf("migration/cloning from '%s' is not allowed: the provided path is invalid", err.Host)
} }

7
services/migrations/migrate.go
View file

@ -81,10 +81,9 @@ func IsMigrateURLAllowed(remoteURL string, doer *user_model.User) error {
err = nil //nolint err = nil //nolint
hostName = u.Host hostName = u.Host
} }
addrList, err := net.LookupIP(hostName)
if err != nil { // some users only use proxy, there is no DNS resolver. it's safe to ignore the LookupIP error
return &models.ErrInvalidCloneAddr{Host: u.Host, NotResolvedIP: true} addrList, _ := net.LookupIP(hostName)
}
var ipAllowed bool var ipAllowed bool
var ipBlocked bool var ipBlocked bool