diff options
| author | David Svantesson <davidsvantesson@gmail.com> | 2019-12-04 02:08:56 +0100 |
|---|---|---|
| committer | techknowlogick <techknowlogick@gitea.io> | 2019-12-03 20:08:56 -0500 |
| commit | bac4b78e0908c0cb01a3842436950c7bcf793cf9 (patch) | |
| tree | a3c80aebb1ca69bf6e518b881229158dccf4ddd7 /modules/auth | |
| parent | 6460284085b0b416d61c57d729d47e932ac05efe (diff) | |
| download | gitea-bac4b78e0908c0cb01a3842436950c7bcf793cf9.tar.gz gitea-bac4b78e0908c0cb01a3842436950c7bcf793cf9.zip | |
Branch protection: Possibility to not use whitelist but allow anyone with write access (#9055)
* Possibility to not use whitelist but allow anyone with write access
* fix existing test
* rename migration function
* Try to give a better name for migration step
* Clear settings if higher level setting is not set
* Move official reviews to db instead of counting approvals each time
* migration
* fix
* fix migration
* fix migration
* Remove NOT NULL from EnableWhitelist as migration isn't possible
* Fix migration, reviews are connected to issues.
* Fix SQL query issues in GetReviewersByPullID.
* Simplify function GetReviewersByIssueID
* Handle reviewers that has been deleted
* Ensure reviews for test is in a well defined order
* Only clear and set official reviews when it is an approve or reject.
Diffstat (limited to 'modules/auth')
| -rw-r--r-- | modules/auth/repo_form.go | 27 |
1 files changed, 14 insertions, 13 deletions
diff --git a/modules/auth/repo_form.go b/modules/auth/repo_form.go index e3f6410c37..26ed2bb692 100644 --- a/modules/auth/repo_form.go +++ b/modules/auth/repo_form.go @@ -157,19 +157,20 @@ func (f *RepoSettingForm) Validate(ctx *macaron.Context, errs binding.Errors) bi // ProtectBranchForm form for changing protected branch settings type ProtectBranchForm struct { - Protected bool - EnableWhitelist bool - WhitelistUsers string - WhitelistTeams string - WhitelistDeployKeys bool - EnableMergeWhitelist bool - MergeWhitelistUsers string - MergeWhitelistTeams string - EnableStatusCheck bool `xorm:"NOT NULL DEFAULT false"` - StatusCheckContexts []string - RequiredApprovals int64 - ApprovalsWhitelistUsers string - ApprovalsWhitelistTeams string + Protected bool + EnablePush string + WhitelistUsers string + WhitelistTeams string + WhitelistDeployKeys bool + EnableMergeWhitelist bool + MergeWhitelistUsers string + MergeWhitelistTeams string + EnableStatusCheck bool `xorm:"NOT NULL DEFAULT false"` + StatusCheckContexts []string + RequiredApprovals int64 + EnableApprovalsWhitelist bool + ApprovalsWhitelistUsers string + ApprovalsWhitelistTeams string } // Validate validates the fields |