Skip to content

Commit

Permalink
Allow http redirects (#400)
Browse files Browse the repository at this point in the history
* Fix HTTP Header redirects causing the checker to fail a domain

* Slightly clarify comment

* Formatting

---------

Co-authored-by: JanHoefelmeyer <[email protected]>
  • Loading branch information
JanHoefelmeyer and JanHoefelmeyer authored Jul 7, 2023
1 parent b423eed commit aeeb169
Showing 1 changed file with 4 additions and 1 deletion.
5 changes: 4 additions & 1 deletion cmd/csaf_checker/rules.go
Original file line number Diff line number Diff line change
Expand Up @@ -166,8 +166,11 @@ func (p *processor) eval(requirement int) bool {

case 5:
return !p.badAmberRedPermissions.hasErrors()
// Currently, only domains using HTTP-Header redirects are checked.
// A domain reaching evaluation will only have HTTP-Header redirects if any,
// and thus requirement 6 will always be fullfilled.
case 6:
return len(p.redirects) == 0
return true
case 7:
return !p.badProviderMetadata.hasErrors()
case 8:
Expand Down

0 comments on commit aeeb169

Please sign in to comment.