mirror of
				https://github.com/go-gitea/gitea.git
				synced 2025-11-03 08:02:36 +09:00 
			
		
		
		
	## Changes
- Adds the following high level access scopes, each with `read` and
`write` levels:
    - `activitypub`
    - `admin` (hidden if user is not a site admin)
    - `misc`
    - `notification`
    - `organization`
    - `package`
    - `issue`
    - `repository`
    - `user`
- Adds new middleware function `tokenRequiresScopes()` in addition to
`reqToken()`
  -  `tokenRequiresScopes()` is used for each high-level api section
- _if_ a scoped token is present, checks that the required scope is
included based on the section and HTTP method
  - `reqToken()` is used for individual routes
- checks that required authentication is present (but does not check
scope levels as this will already have been handled by
`tokenRequiresScopes()`
- Adds migration to convert old scoped access tokens to the new set of
scopes
- Updates the user interface for scope selection
### User interface example
<img width="903" alt="Screen Shot 2023-05-31 at 1 56 55 PM"
src="https://github.com/go-gitea/gitea/assets/23248839/654766ec-2143-4f59-9037-3b51600e32f3">
<img width="917" alt="Screen Shot 2023-05-31 at 1 56 43 PM"
src="https://github.com/go-gitea/gitea/assets/23248839/1ad64081-012c-4a73-b393-66b30352654c">
## tokenRequiresScopes  Design Decision
- `tokenRequiresScopes()` was added to more reliably cover api routes.
For an incoming request, this function uses the given scope category
(say `AccessTokenScopeCategoryOrganization`) and the HTTP method (say
`DELETE`) and verifies that any scoped tokens in use include
`delete:organization`.
- `reqToken()` is used to enforce auth for individual routes that
require it. If a scoped token is not present for a request,
`tokenRequiresScopes()` will not return an error
## TODO
- [x] Alphabetize scope categories
- [x] Change 'public repos only' to a radio button (private vs public).
Also expand this to organizations
- [X] Disable token creation if no scopes selected. Alternatively, show
warning
- [x] `reqToken()` is missing from many `POST/DELETE` routes in the api.
`tokenRequiresScopes()` only checks that a given token has the correct
scope, `reqToken()` must be used to check that a token (or some other
auth) is present.
   -  _This should be addressed in this PR_
- [x] The migration should be reviewed very carefully in order to
minimize access changes to existing user tokens.
   - _This should be addressed in this PR_
- [x] Link to api to swagger documentation, clarify what
read/write/delete levels correspond to
- [x] Review cases where more than one scope is needed as this directly
deviates from the api definition.
   - _This should be addressed in this PR_
   - For example: 
   ```go
	m.Group("/users/{username}/orgs", func() {
		m.Get("", reqToken(), org.ListUserOrgs)
		m.Get("/{org}/permissions", reqToken(), org.GetUserOrgsPermissions)
}, tokenRequiresScopes(auth_model.AccessTokenScopeCategoryUser,
auth_model.AccessTokenScopeCategoryOrganization),
context_service.UserAssignmentAPI())
   ```
## Future improvements
- [ ] Add required scopes to swagger documentation
- [ ] Redesign `reqToken()` to be opt-out rather than opt-in
- [ ] Subdivide scopes like `repository`
- [ ] Once a token is created, if it has no scopes, we should display
text instead of an empty bullet point
- [ ] If the 'public repos only' option is selected, should read
categories be selected by default
Closes #24501
Closes #24799
Co-authored-by: Jonathan Tran <jon@allspice.io>
Co-authored-by: Kyle D <kdumontnu@gmail.com>
Co-authored-by: silverwind <me@silverwind.io>
		
	
		
			
				
	
	
		
			359 lines
		
	
	
		
			16 KiB
		
	
	
	
		
			Go
		
	
	
	
	
	
			
		
		
	
	
			359 lines
		
	
	
		
			16 KiB
		
	
	
	
		
			Go
		
	
	
	
	
	
// Copyright 2019 The Gitea Authors. All rights reserved.
 | 
						|
// SPDX-License-Identifier: MIT
 | 
						|
 | 
						|
package integration
 | 
						|
 | 
						|
import (
 | 
						|
	"fmt"
 | 
						|
	"net/http"
 | 
						|
	"net/url"
 | 
						|
	"testing"
 | 
						|
 | 
						|
	auth_model "code.gitea.io/gitea/models/auth"
 | 
						|
	"code.gitea.io/gitea/models/db"
 | 
						|
	repo_model "code.gitea.io/gitea/models/repo"
 | 
						|
	unit_model "code.gitea.io/gitea/models/unit"
 | 
						|
	"code.gitea.io/gitea/models/unittest"
 | 
						|
	user_model "code.gitea.io/gitea/models/user"
 | 
						|
	api "code.gitea.io/gitea/modules/structs"
 | 
						|
 | 
						|
	"github.com/stretchr/testify/assert"
 | 
						|
)
 | 
						|
 | 
						|
// getRepoEditOptionFromRepo gets the options for an existing repo exactly as is
 | 
						|
func getRepoEditOptionFromRepo(repo *repo_model.Repository) *api.EditRepoOption {
 | 
						|
	name := repo.Name
 | 
						|
	description := repo.Description
 | 
						|
	website := repo.Website
 | 
						|
	private := repo.IsPrivate
 | 
						|
	hasIssues := false
 | 
						|
	var internalTracker *api.InternalTracker
 | 
						|
	var externalTracker *api.ExternalTracker
 | 
						|
	if unit, err := repo.GetUnit(db.DefaultContext, unit_model.TypeIssues); err == nil {
 | 
						|
		config := unit.IssuesConfig()
 | 
						|
		hasIssues = true
 | 
						|
		internalTracker = &api.InternalTracker{
 | 
						|
			EnableTimeTracker:                config.EnableTimetracker,
 | 
						|
			AllowOnlyContributorsToTrackTime: config.AllowOnlyContributorsToTrackTime,
 | 
						|
			EnableIssueDependencies:          config.EnableDependencies,
 | 
						|
		}
 | 
						|
	} else if unit, err := repo.GetUnit(db.DefaultContext, unit_model.TypeExternalTracker); err == nil {
 | 
						|
		config := unit.ExternalTrackerConfig()
 | 
						|
		hasIssues = true
 | 
						|
		externalTracker = &api.ExternalTracker{
 | 
						|
			ExternalTrackerURL:           config.ExternalTrackerURL,
 | 
						|
			ExternalTrackerFormat:        config.ExternalTrackerFormat,
 | 
						|
			ExternalTrackerStyle:         config.ExternalTrackerStyle,
 | 
						|
			ExternalTrackerRegexpPattern: config.ExternalTrackerRegexpPattern,
 | 
						|
		}
 | 
						|
	}
 | 
						|
	hasWiki := false
 | 
						|
	var externalWiki *api.ExternalWiki
 | 
						|
	if _, err := repo.GetUnit(db.DefaultContext, unit_model.TypeWiki); err == nil {
 | 
						|
		hasWiki = true
 | 
						|
	} else if unit, err := repo.GetUnit(db.DefaultContext, unit_model.TypeExternalWiki); err == nil {
 | 
						|
		hasWiki = true
 | 
						|
		config := unit.ExternalWikiConfig()
 | 
						|
		externalWiki = &api.ExternalWiki{
 | 
						|
			ExternalWikiURL: config.ExternalWikiURL,
 | 
						|
		}
 | 
						|
	}
 | 
						|
	defaultBranch := repo.DefaultBranch
 | 
						|
	hasPullRequests := false
 | 
						|
	ignoreWhitespaceConflicts := false
 | 
						|
	allowMerge := false
 | 
						|
	allowRebase := false
 | 
						|
	allowRebaseMerge := false
 | 
						|
	allowSquash := false
 | 
						|
	if unit, err := repo.GetUnit(db.DefaultContext, unit_model.TypePullRequests); err == nil {
 | 
						|
		config := unit.PullRequestsConfig()
 | 
						|
		hasPullRequests = true
 | 
						|
		ignoreWhitespaceConflicts = config.IgnoreWhitespaceConflicts
 | 
						|
		allowMerge = config.AllowMerge
 | 
						|
		allowRebase = config.AllowRebase
 | 
						|
		allowRebaseMerge = config.AllowRebaseMerge
 | 
						|
		allowSquash = config.AllowSquash
 | 
						|
	}
 | 
						|
	archived := repo.IsArchived
 | 
						|
	return &api.EditRepoOption{
 | 
						|
		Name:                      &name,
 | 
						|
		Description:               &description,
 | 
						|
		Website:                   &website,
 | 
						|
		Private:                   &private,
 | 
						|
		HasIssues:                 &hasIssues,
 | 
						|
		ExternalTracker:           externalTracker,
 | 
						|
		InternalTracker:           internalTracker,
 | 
						|
		HasWiki:                   &hasWiki,
 | 
						|
		ExternalWiki:              externalWiki,
 | 
						|
		DefaultBranch:             &defaultBranch,
 | 
						|
		HasPullRequests:           &hasPullRequests,
 | 
						|
		IgnoreWhitespaceConflicts: &ignoreWhitespaceConflicts,
 | 
						|
		AllowMerge:                &allowMerge,
 | 
						|
		AllowRebase:               &allowRebase,
 | 
						|
		AllowRebaseMerge:          &allowRebaseMerge,
 | 
						|
		AllowSquash:               &allowSquash,
 | 
						|
		Archived:                  &archived,
 | 
						|
	}
 | 
						|
}
 | 
						|
 | 
						|
// getNewRepoEditOption Gets the options to change everything about an existing repo by adding to strings or changing
 | 
						|
// the boolean
 | 
						|
func getNewRepoEditOption(opts *api.EditRepoOption) *api.EditRepoOption {
 | 
						|
	// Gives a new property to everything
 | 
						|
	name := *opts.Name + "renamed"
 | 
						|
	description := "new description"
 | 
						|
	website := "http://wwww.newwebsite.com"
 | 
						|
	private := !*opts.Private
 | 
						|
	hasIssues := !*opts.HasIssues
 | 
						|
	hasWiki := !*opts.HasWiki
 | 
						|
	defaultBranch := "master"
 | 
						|
	hasPullRequests := !*opts.HasPullRequests
 | 
						|
	ignoreWhitespaceConflicts := !*opts.IgnoreWhitespaceConflicts
 | 
						|
	allowMerge := !*opts.AllowMerge
 | 
						|
	allowRebase := !*opts.AllowRebase
 | 
						|
	allowRebaseMerge := !*opts.AllowRebaseMerge
 | 
						|
	allowSquash := !*opts.AllowSquash
 | 
						|
	archived := !*opts.Archived
 | 
						|
 | 
						|
	return &api.EditRepoOption{
 | 
						|
		Name:                      &name,
 | 
						|
		Description:               &description,
 | 
						|
		Website:                   &website,
 | 
						|
		Private:                   &private,
 | 
						|
		DefaultBranch:             &defaultBranch,
 | 
						|
		HasIssues:                 &hasIssues,
 | 
						|
		HasWiki:                   &hasWiki,
 | 
						|
		HasPullRequests:           &hasPullRequests,
 | 
						|
		IgnoreWhitespaceConflicts: &ignoreWhitespaceConflicts,
 | 
						|
		AllowMerge:                &allowMerge,
 | 
						|
		AllowRebase:               &allowRebase,
 | 
						|
		AllowRebaseMerge:          &allowRebaseMerge,
 | 
						|
		AllowSquash:               &allowSquash,
 | 
						|
		Archived:                  &archived,
 | 
						|
	}
 | 
						|
}
 | 
						|
 | 
						|
func TestAPIRepoEdit(t *testing.T) {
 | 
						|
	onGiteaRun(t, func(t *testing.T, u *url.URL) {
 | 
						|
		bFalse, bTrue := false, true
 | 
						|
 | 
						|
		user2 := unittest.AssertExistsAndLoadBean(t, &user_model.User{ID: 2})         // owner of the repo1 & repo16
 | 
						|
		user3 := unittest.AssertExistsAndLoadBean(t, &user_model.User{ID: 3})         // owner of the repo3, is an org
 | 
						|
		user4 := unittest.AssertExistsAndLoadBean(t, &user_model.User{ID: 4})         // owner of neither repos
 | 
						|
		repo1 := unittest.AssertExistsAndLoadBean(t, &repo_model.Repository{ID: 1})   // public repo
 | 
						|
		repo3 := unittest.AssertExistsAndLoadBean(t, &repo_model.Repository{ID: 3})   // public repo
 | 
						|
		repo15 := unittest.AssertExistsAndLoadBean(t, &repo_model.Repository{ID: 15}) // empty repo
 | 
						|
		repo16 := unittest.AssertExistsAndLoadBean(t, &repo_model.Repository{ID: 16}) // private repo
 | 
						|
 | 
						|
		// Get user2's token
 | 
						|
		session := loginUser(t, user2.Name)
 | 
						|
		token2 := getTokenForLoggedInUser(t, session, auth_model.AccessTokenScopeWriteRepository)
 | 
						|
		// Get user4's token
 | 
						|
		session = loginUser(t, user4.Name)
 | 
						|
		token4 := getTokenForLoggedInUser(t, session, auth_model.AccessTokenScopeWriteRepository)
 | 
						|
 | 
						|
		// Test editing a repo1 which user2 owns, changing name and many properties
 | 
						|
		origRepoEditOption := getRepoEditOptionFromRepo(repo1)
 | 
						|
		repoEditOption := getNewRepoEditOption(origRepoEditOption)
 | 
						|
		url := fmt.Sprintf("/api/v1/repos/%s/%s?token=%s", user2.Name, repo1.Name, token2)
 | 
						|
		req := NewRequestWithJSON(t, "PATCH", url, &repoEditOption)
 | 
						|
		resp := MakeRequest(t, req, http.StatusOK)
 | 
						|
		var repo api.Repository
 | 
						|
		DecodeJSON(t, resp, &repo)
 | 
						|
		assert.NotNil(t, repo)
 | 
						|
		// check response
 | 
						|
		assert.Equal(t, *repoEditOption.Name, repo.Name)
 | 
						|
		assert.Equal(t, *repoEditOption.Description, repo.Description)
 | 
						|
		assert.Equal(t, *repoEditOption.Website, repo.Website)
 | 
						|
		assert.Equal(t, *repoEditOption.Archived, repo.Archived)
 | 
						|
		// check repo1 from database
 | 
						|
		repo1edited := unittest.AssertExistsAndLoadBean(t, &repo_model.Repository{ID: 1})
 | 
						|
		repo1editedOption := getRepoEditOptionFromRepo(repo1edited)
 | 
						|
		assert.Equal(t, *repoEditOption.Name, *repo1editedOption.Name)
 | 
						|
		assert.Equal(t, *repoEditOption.Description, *repo1editedOption.Description)
 | 
						|
		assert.Equal(t, *repoEditOption.Website, *repo1editedOption.Website)
 | 
						|
		assert.Equal(t, *repoEditOption.Archived, *repo1editedOption.Archived)
 | 
						|
		assert.Equal(t, *repoEditOption.Private, *repo1editedOption.Private)
 | 
						|
		assert.Equal(t, *repoEditOption.HasWiki, *repo1editedOption.HasWiki)
 | 
						|
 | 
						|
		// Test editing repo1 to use internal issue and wiki (default)
 | 
						|
		*repoEditOption.HasIssues = true
 | 
						|
		repoEditOption.ExternalTracker = nil
 | 
						|
		repoEditOption.InternalTracker = &api.InternalTracker{
 | 
						|
			EnableTimeTracker:                false,
 | 
						|
			AllowOnlyContributorsToTrackTime: false,
 | 
						|
			EnableIssueDependencies:          false,
 | 
						|
		}
 | 
						|
		*repoEditOption.HasWiki = true
 | 
						|
		repoEditOption.ExternalWiki = nil
 | 
						|
		url = fmt.Sprintf("/api/v1/repos/%s/%s?token=%s", user2.Name, *repoEditOption.Name, token2)
 | 
						|
		req = NewRequestWithJSON(t, "PATCH", url, &repoEditOption)
 | 
						|
		resp = MakeRequest(t, req, http.StatusOK)
 | 
						|
		DecodeJSON(t, resp, &repo)
 | 
						|
		assert.NotNil(t, repo)
 | 
						|
		// check repo1 was written to database
 | 
						|
		repo1edited = unittest.AssertExistsAndLoadBean(t, &repo_model.Repository{ID: 1})
 | 
						|
		repo1editedOption = getRepoEditOptionFromRepo(repo1edited)
 | 
						|
		assert.True(t, *repo1editedOption.HasIssues)
 | 
						|
		assert.Nil(t, repo1editedOption.ExternalTracker)
 | 
						|
		assert.Equal(t, *repo1editedOption.InternalTracker, *repoEditOption.InternalTracker)
 | 
						|
		assert.True(t, *repo1editedOption.HasWiki)
 | 
						|
		assert.Nil(t, repo1editedOption.ExternalWiki)
 | 
						|
 | 
						|
		// Test editing repo1 to use external issue and wiki
 | 
						|
		repoEditOption.ExternalTracker = &api.ExternalTracker{
 | 
						|
			ExternalTrackerURL:    "http://www.somewebsite.com",
 | 
						|
			ExternalTrackerFormat: "http://www.somewebsite.com/{user}/{repo}?issue={index}",
 | 
						|
			ExternalTrackerStyle:  "alphanumeric",
 | 
						|
		}
 | 
						|
		repoEditOption.ExternalWiki = &api.ExternalWiki{
 | 
						|
			ExternalWikiURL: "http://www.somewebsite.com",
 | 
						|
		}
 | 
						|
		req = NewRequestWithJSON(t, "PATCH", url, &repoEditOption)
 | 
						|
		resp = MakeRequest(t, req, http.StatusOK)
 | 
						|
		DecodeJSON(t, resp, &repo)
 | 
						|
		assert.NotNil(t, repo)
 | 
						|
		// check repo1 was written to database
 | 
						|
		repo1edited = unittest.AssertExistsAndLoadBean(t, &repo_model.Repository{ID: 1})
 | 
						|
		repo1editedOption = getRepoEditOptionFromRepo(repo1edited)
 | 
						|
		assert.True(t, *repo1editedOption.HasIssues)
 | 
						|
		assert.Equal(t, *repo1editedOption.ExternalTracker, *repoEditOption.ExternalTracker)
 | 
						|
		assert.True(t, *repo1editedOption.HasWiki)
 | 
						|
		assert.Equal(t, *repo1editedOption.ExternalWiki, *repoEditOption.ExternalWiki)
 | 
						|
 | 
						|
		repoEditOption.ExternalTracker.ExternalTrackerStyle = "regexp"
 | 
						|
		repoEditOption.ExternalTracker.ExternalTrackerRegexpPattern = `(\d+)`
 | 
						|
		req = NewRequestWithJSON(t, "PATCH", url, &repoEditOption)
 | 
						|
		resp = MakeRequest(t, req, http.StatusOK)
 | 
						|
		DecodeJSON(t, resp, &repo)
 | 
						|
		assert.NotNil(t, repo)
 | 
						|
		repo1edited = unittest.AssertExistsAndLoadBean(t, &repo_model.Repository{ID: 1})
 | 
						|
		repo1editedOption = getRepoEditOptionFromRepo(repo1edited)
 | 
						|
		assert.True(t, *repo1editedOption.HasIssues)
 | 
						|
		assert.Equal(t, *repo1editedOption.ExternalTracker, *repoEditOption.ExternalTracker)
 | 
						|
 | 
						|
		// Do some tests with invalid URL for external tracker and wiki
 | 
						|
		repoEditOption.ExternalTracker.ExternalTrackerURL = "htp://www.somewebsite.com"
 | 
						|
		req = NewRequestWithJSON(t, "PATCH", url, &repoEditOption)
 | 
						|
		MakeRequest(t, req, http.StatusUnprocessableEntity)
 | 
						|
		repoEditOption.ExternalTracker.ExternalTrackerURL = "http://www.somewebsite.com"
 | 
						|
		repoEditOption.ExternalTracker.ExternalTrackerFormat = "http://www.somewebsite.com/{user/{repo}?issue={index}"
 | 
						|
		req = NewRequestWithJSON(t, "PATCH", url, &repoEditOption)
 | 
						|
		MakeRequest(t, req, http.StatusUnprocessableEntity)
 | 
						|
		repoEditOption.ExternalTracker.ExternalTrackerFormat = "http://www.somewebsite.com/{user}/{repo}?issue={index}"
 | 
						|
		repoEditOption.ExternalWiki.ExternalWikiURL = "htp://www.somewebsite.com"
 | 
						|
		req = NewRequestWithJSON(t, "PATCH", url, &repoEditOption)
 | 
						|
		MakeRequest(t, req, http.StatusUnprocessableEntity)
 | 
						|
 | 
						|
		// Test small repo change through API with issue and wiki option not set; They shall not be touched.
 | 
						|
		*repoEditOption.Description = "small change"
 | 
						|
		repoEditOption.HasIssues = nil
 | 
						|
		repoEditOption.ExternalTracker = nil
 | 
						|
		repoEditOption.HasWiki = nil
 | 
						|
		repoEditOption.ExternalWiki = nil
 | 
						|
		req = NewRequestWithJSON(t, "PATCH", url, &repoEditOption)
 | 
						|
		resp = MakeRequest(t, req, http.StatusOK)
 | 
						|
		DecodeJSON(t, resp, &repo)
 | 
						|
		assert.NotNil(t, repo)
 | 
						|
		// check repo1 was written to database
 | 
						|
		repo1edited = unittest.AssertExistsAndLoadBean(t, &repo_model.Repository{ID: 1})
 | 
						|
		repo1editedOption = getRepoEditOptionFromRepo(repo1edited)
 | 
						|
		assert.Equal(t, *repo1editedOption.Description, *repoEditOption.Description)
 | 
						|
		assert.True(t, *repo1editedOption.HasIssues)
 | 
						|
		assert.NotNil(t, *repo1editedOption.ExternalTracker)
 | 
						|
		assert.True(t, *repo1editedOption.HasWiki)
 | 
						|
		assert.NotNil(t, *repo1editedOption.ExternalWiki)
 | 
						|
 | 
						|
		// reset repo in db
 | 
						|
		url = fmt.Sprintf("/api/v1/repos/%s/%s?token=%s", user2.Name, *repoEditOption.Name, token2)
 | 
						|
		req = NewRequestWithJSON(t, "PATCH", url, &origRepoEditOption)
 | 
						|
		_ = MakeRequest(t, req, http.StatusOK)
 | 
						|
 | 
						|
		// Test editing a non-existing repo
 | 
						|
		name := "repodoesnotexist"
 | 
						|
		url = fmt.Sprintf("/api/v1/repos/%s/%s?token=%s", user2.Name, name, token2)
 | 
						|
		req = NewRequestWithJSON(t, "PATCH", url, &api.EditRepoOption{Name: &name})
 | 
						|
		_ = MakeRequest(t, req, http.StatusNotFound)
 | 
						|
 | 
						|
		// Test editing repo16 by user4 who does not have write access
 | 
						|
		origRepoEditOption = getRepoEditOptionFromRepo(repo16)
 | 
						|
		repoEditOption = getNewRepoEditOption(origRepoEditOption)
 | 
						|
		url = fmt.Sprintf("/api/v1/repos/%s/%s?token=%s", user2.Name, repo16.Name, token4)
 | 
						|
		req = NewRequestWithJSON(t, "PATCH", url, &repoEditOption)
 | 
						|
		MakeRequest(t, req, http.StatusNotFound)
 | 
						|
 | 
						|
		// Tests a repo with no token given so will fail
 | 
						|
		origRepoEditOption = getRepoEditOptionFromRepo(repo16)
 | 
						|
		repoEditOption = getNewRepoEditOption(origRepoEditOption)
 | 
						|
		url = fmt.Sprintf("/api/v1/repos/%s/%s", user2.Name, repo16.Name)
 | 
						|
		req = NewRequestWithJSON(t, "PATCH", url, &repoEditOption)
 | 
						|
		_ = MakeRequest(t, req, http.StatusNotFound)
 | 
						|
 | 
						|
		// Test using access token for a private repo that the user of the token owns
 | 
						|
		origRepoEditOption = getRepoEditOptionFromRepo(repo16)
 | 
						|
		repoEditOption = getNewRepoEditOption(origRepoEditOption)
 | 
						|
		url = fmt.Sprintf("/api/v1/repos/%s/%s?token=%s", user2.Name, repo16.Name, token2)
 | 
						|
		req = NewRequestWithJSON(t, "PATCH", url, &repoEditOption)
 | 
						|
		_ = MakeRequest(t, req, http.StatusOK)
 | 
						|
		// reset repo in db
 | 
						|
		url = fmt.Sprintf("/api/v1/repos/%s/%s?token=%s", user2.Name, *repoEditOption.Name, token2)
 | 
						|
		req = NewRequestWithJSON(t, "PATCH", url, &origRepoEditOption)
 | 
						|
		_ = MakeRequest(t, req, http.StatusOK)
 | 
						|
 | 
						|
		// Test making a repo public that is private
 | 
						|
		repo16 = unittest.AssertExistsAndLoadBean(t, &repo_model.Repository{ID: 16})
 | 
						|
		assert.True(t, repo16.IsPrivate)
 | 
						|
		repoEditOption = &api.EditRepoOption{
 | 
						|
			Private: &bFalse,
 | 
						|
		}
 | 
						|
		url = fmt.Sprintf("/api/v1/repos/%s/%s?token=%s", user2.Name, repo16.Name, token2)
 | 
						|
		req = NewRequestWithJSON(t, "PATCH", url, &repoEditOption)
 | 
						|
		_ = MakeRequest(t, req, http.StatusOK)
 | 
						|
		repo16 = unittest.AssertExistsAndLoadBean(t, &repo_model.Repository{ID: 16})
 | 
						|
		assert.False(t, repo16.IsPrivate)
 | 
						|
		// Make it private again
 | 
						|
		repoEditOption.Private = &bTrue
 | 
						|
		req = NewRequestWithJSON(t, "PATCH", url, &repoEditOption)
 | 
						|
		_ = MakeRequest(t, req, http.StatusOK)
 | 
						|
 | 
						|
		// Test to change empty repo
 | 
						|
		assert.False(t, repo15.IsArchived)
 | 
						|
		url = fmt.Sprintf("/api/v1/repos/%s/%s?token=%s", user2.Name, repo15.Name, token2)
 | 
						|
		req = NewRequestWithJSON(t, "PATCH", url, &api.EditRepoOption{
 | 
						|
			Archived: &bTrue,
 | 
						|
		})
 | 
						|
		_ = MakeRequest(t, req, http.StatusOK)
 | 
						|
		repo15 = unittest.AssertExistsAndLoadBean(t, &repo_model.Repository{ID: 15})
 | 
						|
		assert.True(t, repo15.IsArchived)
 | 
						|
		req = NewRequestWithJSON(t, "PATCH", url, &api.EditRepoOption{
 | 
						|
			Archived: &bFalse,
 | 
						|
		})
 | 
						|
		_ = MakeRequest(t, req, http.StatusOK)
 | 
						|
 | 
						|
		// Test using org repo "user3/repo3" where user2 is a collaborator
 | 
						|
		origRepoEditOption = getRepoEditOptionFromRepo(repo3)
 | 
						|
		repoEditOption = getNewRepoEditOption(origRepoEditOption)
 | 
						|
		url = fmt.Sprintf("/api/v1/repos/%s/%s?token=%s", user3.Name, repo3.Name, token2)
 | 
						|
		req = NewRequestWithJSON(t, "PATCH", url, &repoEditOption)
 | 
						|
		MakeRequest(t, req, http.StatusOK)
 | 
						|
		// reset repo in db
 | 
						|
		url = fmt.Sprintf("/api/v1/repos/%s/%s?token=%s", user3.Name, *repoEditOption.Name, token2)
 | 
						|
		req = NewRequestWithJSON(t, "PATCH", url, &origRepoEditOption)
 | 
						|
		_ = MakeRequest(t, req, http.StatusOK)
 | 
						|
 | 
						|
		// Test using org repo "user3/repo3" with no user token
 | 
						|
		origRepoEditOption = getRepoEditOptionFromRepo(repo3)
 | 
						|
		repoEditOption = getNewRepoEditOption(origRepoEditOption)
 | 
						|
		url = fmt.Sprintf("/api/v1/repos/%s/%s", user3.Name, repo3.Name)
 | 
						|
		req = NewRequestWithJSON(t, "PATCH", url, &repoEditOption)
 | 
						|
		MakeRequest(t, req, http.StatusNotFound)
 | 
						|
 | 
						|
		// Test using repo "user2/repo1" where user4 is a NOT collaborator
 | 
						|
		origRepoEditOption = getRepoEditOptionFromRepo(repo1)
 | 
						|
		repoEditOption = getNewRepoEditOption(origRepoEditOption)
 | 
						|
		url = fmt.Sprintf("/api/v1/repos/%s/%s?token=%s", user2.Name, repo1.Name, token4)
 | 
						|
		req = NewRequestWithJSON(t, "PATCH", url, &repoEditOption)
 | 
						|
		MakeRequest(t, req, http.StatusForbidden)
 | 
						|
	})
 | 
						|
}
 |