Compare commits
No commits in common. "2d0a62dc459274ced1cb24cbf142a5bd2b3b6946" and "0a501582394c63581d52dacb19777e3bf3aeaf89" have entirely different histories.
2d0a62dc45
...
0a50158239
|
@ -27,13 +27,6 @@ gitlab:
|
|||
# A list of the user ids to expose their personal projects in the filesystem.
|
||||
user_ids: []
|
||||
|
||||
# Set how archived projects are handled.
|
||||
# If set to "show", it will add them to the filesystem and treat them like any other repository
|
||||
# If set to "hide", it will add them to the filesystem, but prefix the symlink with a "."
|
||||
# If set to "ignore", it will make them absent from the filesystem
|
||||
# Default to "hide"
|
||||
archived_project_handling: hide
|
||||
|
||||
# If set to true, the user the api token belongs to will automatically be added to the list of users exposed by the filesystem.
|
||||
include_current_user: true
|
||||
|
||||
|
|
|
@ -36,13 +36,12 @@ func LoadConfig(configPath string) (*Config, error) {
|
|||
MountOptions: "nodev,nosuid",
|
||||
},
|
||||
Gitlab: gitlab.GitlabClientConfig{
|
||||
URL: "https://gitlab.com",
|
||||
Token: "",
|
||||
PullMethod: "http",
|
||||
GroupIDs: []int{9970},
|
||||
UserIDs: []int{},
|
||||
ArchivedProjectHandling: "hide",
|
||||
IncludeCurrentUser: true,
|
||||
URL: "https://gitlab.com",
|
||||
Token: "",
|
||||
PullMethod: "http",
|
||||
GroupIDs: []int{9970},
|
||||
UserIDs: []int{},
|
||||
IncludeCurrentUser: true,
|
||||
},
|
||||
Git: git.GitClientParam{
|
||||
CloneLocation: defaultCloneLocation,
|
||||
|
@ -86,10 +85,5 @@ func MakeGitlabConfig(config *Config) (*gitlab.GitlabClientConfig, error) {
|
|||
return nil, fmt.Errorf("pull_method must be either \"%v\" or \"%v\"", gitlab.PullMethodHTTP, gitlab.PullMethodSSH)
|
||||
}
|
||||
|
||||
// parse archive_handing
|
||||
if config.Gitlab.ArchivedProjectHandling != gitlab.ArchivedProjectShow && config.Gitlab.ArchivedProjectHandling != gitlab.ArchivedProjectHide && config.Gitlab.ArchivedProjectHandling != gitlab.ArchivedProjectIgnore {
|
||||
return nil, fmt.Errorf("pull_method must be either \"%v\", \"%v\" or \"%v\"", gitlab.ArchivedProjectShow, gitlab.ArchivedProjectHide, gitlab.ArchivedProjectIgnore)
|
||||
}
|
||||
|
||||
return &config.Gitlab, nil
|
||||
}
|
||||
|
|
|
@ -22,13 +22,12 @@ func TestLoadConfig(t *testing.T) {
|
|||
MountOptions: "nodev",
|
||||
},
|
||||
Gitlab: gitlab.GitlabClientConfig{
|
||||
URL: "https://example.com",
|
||||
Token: "12345",
|
||||
PullMethod: "ssh",
|
||||
GroupIDs: []int{123},
|
||||
UserIDs: []int{456},
|
||||
ArchivedProjectHandling: "hide",
|
||||
IncludeCurrentUser: true,
|
||||
URL: "https://example.com",
|
||||
Token: "12345",
|
||||
PullMethod: "ssh",
|
||||
GroupIDs: []int{123},
|
||||
UserIDs: []int{456},
|
||||
IncludeCurrentUser: true,
|
||||
},
|
||||
Git: git.GitClientParam{
|
||||
CloneLocation: "/tmp/gitlabfs/test/clone",
|
||||
|
@ -119,49 +118,32 @@ func TestMakeGitlabConfig(t *testing.T) {
|
|||
"ValidConfig": {
|
||||
input: &config.Config{
|
||||
Gitlab: gitlab.GitlabClientConfig{
|
||||
URL: "https://gitlab.com",
|
||||
PullMethod: "http",
|
||||
Token: "",
|
||||
GroupIDs: []int{9970},
|
||||
UserIDs: []int{},
|
||||
ArchivedProjectHandling: "hide",
|
||||
IncludeCurrentUser: true,
|
||||
URL: "https://gitlab.com",
|
||||
Token: "",
|
||||
GroupIDs: []int{9970},
|
||||
UserIDs: []int{},
|
||||
IncludeCurrentUser: true,
|
||||
PullMethod: "http",
|
||||
},
|
||||
},
|
||||
expected: &gitlab.GitlabClientConfig{
|
||||
URL: "https://gitlab.com",
|
||||
PullMethod: "http",
|
||||
Token: "",
|
||||
GroupIDs: []int{9970},
|
||||
UserIDs: []int{},
|
||||
ArchivedProjectHandling: "hide",
|
||||
IncludeCurrentUser: true,
|
||||
URL: "https://gitlab.com",
|
||||
Token: "",
|
||||
GroupIDs: []int{9970},
|
||||
UserIDs: []int{},
|
||||
IncludeCurrentUser: true,
|
||||
PullMethod: "http",
|
||||
},
|
||||
},
|
||||
"InvalidPullMethod": {
|
||||
input: &config.Config{
|
||||
Gitlab: gitlab.GitlabClientConfig{
|
||||
URL: "https://gitlab.com",
|
||||
PullMethod: "invalid",
|
||||
Token: "",
|
||||
GroupIDs: []int{9970},
|
||||
UserIDs: []int{},
|
||||
ArchivedProjectHandling: "hide",
|
||||
IncludeCurrentUser: true,
|
||||
},
|
||||
},
|
||||
expected: nil,
|
||||
},
|
||||
"InvalidArchiveHandling": {
|
||||
input: &config.Config{
|
||||
Gitlab: gitlab.GitlabClientConfig{
|
||||
URL: "https://gitlab.com",
|
||||
PullMethod: "http",
|
||||
Token: "",
|
||||
GroupIDs: []int{9970},
|
||||
UserIDs: []int{},
|
||||
IncludeCurrentUser: true,
|
||||
ArchivedProjectHandling: "invalid",
|
||||
URL: "https://gitlab.com",
|
||||
Token: "",
|
||||
GroupIDs: []int{9970},
|
||||
UserIDs: []int{},
|
||||
IncludeCurrentUser: true,
|
||||
PullMethod: "invalid",
|
||||
},
|
||||
},
|
||||
expected: nil,
|
||||
|
|
|
@ -18,7 +18,7 @@ type groupNode struct {
|
|||
|
||||
type GroupSource interface {
|
||||
GetGroupID() uint64
|
||||
InvalidateContentCache()
|
||||
InvalidateCache()
|
||||
}
|
||||
|
||||
// Ensure we are implementing the NodeReaddirer interface
|
||||
|
|
|
@ -41,6 +41,6 @@ func (n *refreshNode) Setattr(ctx context.Context, fh fs.FileHandle, in *fuse.Se
|
|||
}
|
||||
|
||||
func (n *refreshNode) Open(ctx context.Context, flags uint32) (fh fs.FileHandle, fuseFlags uint32, errno syscall.Errno) {
|
||||
n.source.InvalidateContentCache()
|
||||
n.source.InvalidateCache()
|
||||
return nil, 0, 0
|
||||
}
|
||||
|
|
|
@ -4,7 +4,6 @@ import (
|
|||
"fmt"
|
||||
"log/slog"
|
||||
"slices"
|
||||
"sync"
|
||||
|
||||
"github.com/badjware/gitlabfs/fstree"
|
||||
"github.com/xanzy/go-gitlab"
|
||||
|
@ -13,20 +12,15 @@ import (
|
|||
const (
|
||||
PullMethodHTTP = "http"
|
||||
PullMethodSSH = "ssh"
|
||||
|
||||
ArchivedProjectShow = "show"
|
||||
ArchivedProjectHide = "hide"
|
||||
ArchivedProjectIgnore = "ignore"
|
||||
)
|
||||
|
||||
type GitlabClientConfig struct {
|
||||
URL string `yaml:"url,omitempty"`
|
||||
Token string `yaml:"token,omitempty"`
|
||||
GroupIDs []int `yaml:"group_ids,omitempty"`
|
||||
UserIDs []int `yaml:"user_ids,omitempty"`
|
||||
ArchivedProjectHandling string `yaml:"archived_project_handling,omitempty"`
|
||||
IncludeCurrentUser bool `yaml:"include_current_user,omitempty"`
|
||||
PullMethod string `yaml:"pull_method,omitempty"`
|
||||
URL string `yaml:"url,omitempty"`
|
||||
Token string `yaml:"token,omitempty"`
|
||||
GroupIDs []int `yaml:"group_ids,omitempty"`
|
||||
UserIDs []int `yaml:"user_ids,omitempty"`
|
||||
IncludeCurrentUser bool `yaml:"include_current_user,omitempty"`
|
||||
PullMethod string `yaml:"pull_method,omitempty"`
|
||||
}
|
||||
|
||||
type gitlabClient struct {
|
||||
|
@ -40,10 +34,8 @@ type gitlabClient struct {
|
|||
currentUserCache *User
|
||||
|
||||
// API response cache
|
||||
groupCacheMux sync.RWMutex
|
||||
groupCache map[int]*Group
|
||||
userCacheMux sync.RWMutex
|
||||
userCache map[int]*User
|
||||
groupCache map[int]*Group
|
||||
userCache map[int]*User
|
||||
}
|
||||
|
||||
func NewClient(logger *slog.Logger, gitlabUrl string, gitlabToken string, p GitlabClientConfig) (*gitlabClient, error) {
|
||||
|
|
|
@ -16,7 +16,7 @@ type Group struct {
|
|||
|
||||
mux sync.Mutex
|
||||
|
||||
// hold group content
|
||||
// group content
|
||||
childGroups map[string]fstree.GroupSource
|
||||
childProjects map[string]fstree.RepositorySource
|
||||
}
|
||||
|
@ -25,17 +25,15 @@ func (g *Group) GetGroupID() uint64 {
|
|||
return uint64(g.ID)
|
||||
}
|
||||
|
||||
func (g *Group) InvalidateContentCache() {
|
||||
func (g *Group) InvalidateCache() {
|
||||
g.mux.Lock()
|
||||
defer g.mux.Unlock()
|
||||
|
||||
// clear child group from cache
|
||||
g.gitlabClient.groupCacheMux.Lock()
|
||||
for _, childGroup := range g.childGroups {
|
||||
gid := int(childGroup.GetGroupID())
|
||||
delete(g.gitlabClient.groupCache, gid)
|
||||
}
|
||||
g.gitlabClient.groupCacheMux.Unlock()
|
||||
g.childGroups = nil
|
||||
|
||||
// clear child repositories from cache
|
||||
|
@ -45,9 +43,7 @@ func (g *Group) InvalidateContentCache() {
|
|||
func (c *gitlabClient) fetchGroup(gid int) (*Group, error) {
|
||||
// start by searching the cache
|
||||
// TODO: cache invalidation?
|
||||
c.groupCacheMux.RLock()
|
||||
group, found := c.groupCache[gid]
|
||||
c.groupCacheMux.RUnlock()
|
||||
if found {
|
||||
c.logger.Debug("Group cache hit", "gid", gid)
|
||||
return group, nil
|
||||
|
@ -72,9 +68,7 @@ func (c *gitlabClient) fetchGroup(gid int) (*Group, error) {
|
|||
}
|
||||
|
||||
// save in cache
|
||||
c.groupCacheMux.Lock()
|
||||
c.groupCache[gid] = &newGroup
|
||||
c.groupCacheMux.Unlock()
|
||||
|
||||
return &newGroup, nil
|
||||
}
|
||||
|
@ -83,18 +77,15 @@ func (c *gitlabClient) newGroupFromGitlabGroup(gitlabGroup *gitlab.Group) (*Grou
|
|||
gid := gitlabGroup.ID
|
||||
|
||||
// start by searching the cache
|
||||
c.groupCacheMux.RLock()
|
||||
group, found := c.groupCache[gid]
|
||||
c.groupCacheMux.RUnlock()
|
||||
if found {
|
||||
// if found in cache, return the cached reference
|
||||
c.logger.Debug("Group cache hit", "gid", gid)
|
||||
return group, nil
|
||||
} else {
|
||||
c.logger.Debug("Group cache miss; registering group", "gid", gid)
|
||||
}
|
||||
|
||||
// if not found in cache, convert and save to cache now
|
||||
// if not in cache, convert and save to cache now
|
||||
newGroup := Group{
|
||||
ID: gitlabGroup.ID,
|
||||
Name: gitlabGroup.Path,
|
||||
|
@ -106,17 +97,12 @@ func (c *gitlabClient) newGroupFromGitlabGroup(gitlabGroup *gitlab.Group) (*Grou
|
|||
}
|
||||
|
||||
// save in cache
|
||||
c.groupCacheMux.Lock()
|
||||
c.groupCache[gid] = &newGroup
|
||||
c.groupCacheMux.Unlock()
|
||||
|
||||
return &newGroup, nil
|
||||
}
|
||||
|
||||
func (c *gitlabClient) fetchGroupContent(group *Group) (map[string]fstree.GroupSource, map[string]fstree.RepositorySource, error) {
|
||||
// Only a single routine can fetch the group content at the time.
|
||||
// We lock for the whole duration of the function to avoid fetching the same data from the API
|
||||
// multiple times if concurrent calls where to occur.
|
||||
group.mux.Lock()
|
||||
defer group.mux.Unlock()
|
||||
|
||||
|
@ -163,9 +149,7 @@ func (c *gitlabClient) fetchGroupContent(group *Group) (map[string]fstree.GroupS
|
|||
}
|
||||
for _, gitlabProject := range gitlabProjects {
|
||||
project := c.newProjectFromGitlabProject(gitlabProject)
|
||||
if project != nil {
|
||||
childProjects[project.Path] = project
|
||||
}
|
||||
childProjects[project.Name] = &project
|
||||
}
|
||||
if response.CurrentPage >= response.TotalPages {
|
||||
break
|
||||
|
|
|
@ -1,14 +1,12 @@
|
|||
package gitlab
|
||||
|
||||
import (
|
||||
"path"
|
||||
|
||||
"github.com/xanzy/go-gitlab"
|
||||
)
|
||||
|
||||
type Project struct {
|
||||
ID int
|
||||
Path string
|
||||
Name string
|
||||
CloneURL string
|
||||
DefaultBranch string
|
||||
}
|
||||
|
@ -25,14 +23,11 @@ func (p *Project) GetDefaultBranch() string {
|
|||
return p.DefaultBranch
|
||||
}
|
||||
|
||||
func (c *gitlabClient) newProjectFromGitlabProject(project *gitlab.Project) *Project {
|
||||
func (c *gitlabClient) newProjectFromGitlabProject(project *gitlab.Project) Project {
|
||||
// https://godoc.org/github.com/xanzy/go-gitlab#Project
|
||||
if c.ArchivedProjectHandling == ArchivedProjectIgnore && project.Archived {
|
||||
return nil
|
||||
}
|
||||
p := Project{
|
||||
ID: project.ID,
|
||||
Path: project.Path,
|
||||
Name: project.Path,
|
||||
DefaultBranch: project.DefaultBranch,
|
||||
}
|
||||
if p.DefaultBranch == "" {
|
||||
|
@ -43,8 +38,5 @@ func (c *gitlabClient) newProjectFromGitlabProject(project *gitlab.Project) *Pro
|
|||
} else {
|
||||
p.CloneURL = project.HTTPURLToRepo
|
||||
}
|
||||
if c.ArchivedProjectHandling == ArchivedProjectHide && project.Archived {
|
||||
p.Path = path.Join(path.Dir(p.Path), "."+path.Base(p.Path))
|
||||
}
|
||||
return &p
|
||||
return p
|
||||
}
|
||||
|
|
|
@ -14,7 +14,7 @@ type User struct {
|
|||
|
||||
mux sync.Mutex
|
||||
|
||||
// hold user content
|
||||
// user content
|
||||
childProjects map[string]fstree.RepositorySource
|
||||
}
|
||||
|
||||
|
@ -22,7 +22,7 @@ func (u *User) GetGroupID() uint64 {
|
|||
return uint64(u.ID)
|
||||
}
|
||||
|
||||
func (u *User) InvalidateContentCache() {
|
||||
func (u *User) InvalidateCache() {
|
||||
u.mux.Lock()
|
||||
defer u.mux.Unlock()
|
||||
|
||||
|
@ -33,18 +33,15 @@ func (u *User) InvalidateContentCache() {
|
|||
func (c *gitlabClient) fetchUser(uid int) (*User, error) {
|
||||
// start by searching the cache
|
||||
// TODO: cache invalidation?
|
||||
c.userCacheMux.RLock()
|
||||
user, found := c.userCache[uid]
|
||||
c.userCacheMux.RUnlock()
|
||||
if found {
|
||||
// if found in cache, return the cached reference
|
||||
c.logger.Debug("User cache hit", "uid", uid)
|
||||
return user, nil
|
||||
} else {
|
||||
c.logger.Debug("User cache miss", "uid", uid)
|
||||
}
|
||||
|
||||
// If not found in cache, fetch group infos from API
|
||||
// If not in cache, fetch group infos from API
|
||||
gitlabUser, _, err := c.client.Users.GetUser(uid)
|
||||
if err != nil {
|
||||
return nil, fmt.Errorf("failed to fetch user with id %v: %v", uid, err)
|
||||
|
@ -58,7 +55,6 @@ func (c *gitlabClient) fetchUser(uid int) (*User, error) {
|
|||
|
||||
// save in cache
|
||||
c.userCache[uid] = &newUser
|
||||
c.userCacheMux.Unlock()
|
||||
|
||||
return &newUser, nil
|
||||
}
|
||||
|
@ -81,9 +77,6 @@ func (c *gitlabClient) fetchCurrentUser() (*User, error) {
|
|||
}
|
||||
|
||||
func (c *gitlabClient) fetchUserContent(user *User) (map[string]fstree.GroupSource, map[string]fstree.RepositorySource, error) {
|
||||
// Only a single routine can fetch the user content at the time.
|
||||
// We lock for the whole duration of the function to avoid fetching the same data from the API
|
||||
// multiple times if concurrent calls where to occur.
|
||||
user.mux.Lock()
|
||||
defer user.mux.Unlock()
|
||||
|
||||
|
@ -105,9 +98,7 @@ func (c *gitlabClient) fetchUserContent(user *User) (map[string]fstree.GroupSour
|
|||
}
|
||||
for _, gitlabProject := range gitlabProjects {
|
||||
project := c.newProjectFromGitlabProject(gitlabProject)
|
||||
if project != nil {
|
||||
childProjects[project.Path] = project
|
||||
}
|
||||
childProjects[project.Name] = &project
|
||||
}
|
||||
if response.CurrentPage >= response.TotalPages {
|
||||
break
|
||||
|
|
Loading…
Reference in New Issue