Compare commits

..

2 Commits

Author SHA1 Message Date
ea97070fea add support for getting a user name from config
Some checks failed
Declarative Tests / test (push) Successful in 41s
Lint / golangci-lint (push) Failing after 10m37s
2024-10-04 10:35:27 -07:00
dae9c1cf45 fix exception when handling http status 2024-10-04 05:18:32 +00:00
4 changed files with 45 additions and 4 deletions

View File

@ -182,6 +182,11 @@ func (u *User) UseConfig(config data.ConfigurationValueGetter) {
}
func (u *User) ResolveId(ctx context.Context) string {
if u.config != nil {
if configUser, configUserErr := u.config.GetValue("user"); configUserErr == nil && u.Name == "" {
u.Name = configUser.(string)
}
}
return LookupUIDString(u.Name)
}

View File

@ -13,6 +13,7 @@ import (
_ "os"
_ "strings"
"testing"
"decl/internal/data"
)
func TestNewUserResource(t *testing.T) {
@ -77,3 +78,22 @@ func TestCreateUser(t *testing.T) {
applyDeleteErr := u.Apply()
assert.Equal(t, nil, applyDeleteErr)
}
func TestSystemUser(t *testing.T) {
u := NewUser()
u.UseConfig(MockConfigValueGetter(func(key string) (any, error) {
switch key {
case "user":
return "bar", nil
case "group":
return "foo", nil
case "uid":
return "", nil
}
return nil, fmt.Errorf("%w: %s", data.ErrUnknownConfigurationKey, key)
}))
u.ResolveId(context.Background())
assert.Equal(t, "bar", u.Name)
}

View File

@ -91,6 +91,10 @@ func (h *HTTP) Stat() (info fs.FileInfo, err error) {
}
defer response.Body.Close()
if response.StatusCode > 399 {
err = ErrTransportResourceAbsent
return
}
fi := NewHTTPFileInfo(uri)
contentLength := response.Header.Get("Content-Length")
if contentLength != "" {

View File

@ -3,7 +3,7 @@
package transport
import (
_ "errors"
"errors"
_ "fmt"
"net/url"
"net/http"
@ -15,6 +15,10 @@ _ "os"
"context"
)
var (
ErrTransportResourceAbsent error = errors.New("Transport resource is absent")
)
type Handler interface {
URI() *url.URL
ContentType() string
@ -162,11 +166,19 @@ func (r *Reader) AddHeader(name string, value string) {
}
func (r *Reader) Status() string {
return r.handle.(*HTTPReader).GetResponse().Status
if httpReader, ok := r.handle.(*HTTPReader); ok {
return httpReader.GetResponse().Status
}
panic("Unable to get HTTP status from reader")
return ""
}
func (r *Reader) StatusCode() int {
return r.handle.(*HTTPReader).GetResponse().StatusCode
if httpReader, ok := r.handle.(*HTTPReader); ok {
return httpReader.GetResponse().StatusCode
}
panic("Unable to get HTTP status code from reader")
return 0
}
func (r *Reader) Response() *http.Response {