Move password hashers from server/v8 to server/public to fix layering violation

This commit is contained in:
Christopher Speller
2026-03-26 19:54:40 +01:00
parent 5aefff30cf
commit 8cad5b8dc9
20 changed files with 13 additions and 16 deletions
+1 -1
View File
@@ -10,9 +10,9 @@ import (
"strings"
"github.com/mattermost/mattermost/server/public/model"
"github.com/mattermost/mattermost/server/public/password/hashers"
"github.com/mattermost/mattermost/server/public/shared/mlog"
"github.com/mattermost/mattermost/server/public/shared/request"
"github.com/mattermost/mattermost/server/v8/channels/app/password/hashers"
"github.com/mattermost/mattermost/server/v8/channels/app/users"
"github.com/mattermost/mattermost/server/v8/channels/utils"
"github.com/mattermost/mattermost/server/v8/platform/shared/mfa"
+1 -1
View File
@@ -18,7 +18,7 @@ import (
"golang.org/x/crypto/bcrypt"
"github.com/mattermost/mattermost/server/public/model"
"github.com/mattermost/mattermost/server/v8/channels/app/password/hashers"
"github.com/mattermost/mattermost/server/public/password/hashers"
"github.com/mattermost/mattermost/server/v8/einterfaces/mocks"
)
+1 -1
View File
@@ -19,13 +19,13 @@ import (
"golang.org/x/sync/errgroup"
"github.com/mattermost/mattermost/server/public/model"
"github.com/mattermost/mattermost/server/public/password/hashers"
"github.com/mattermost/mattermost/server/public/plugin"
"github.com/mattermost/mattermost/server/public/shared/i18n"
"github.com/mattermost/mattermost/server/public/shared/mlog"
"github.com/mattermost/mattermost/server/public/shared/request"
"github.com/mattermost/mattermost/server/v8/channels/app/email"
"github.com/mattermost/mattermost/server/v8/channels/app/imaging"
"github.com/mattermost/mattermost/server/v8/channels/app/password/hashers"
"github.com/mattermost/mattermost/server/v8/channels/app/users"
"github.com/mattermost/mattermost/server/v8/channels/store"
"github.com/mattermost/mattermost/server/v8/einterfaces"
@@ -16,8 +16,8 @@ import (
"github.com/stretchr/testify/require"
"github.com/mattermost/mattermost/server/public/model"
"github.com/mattermost/mattermost/server/public/password/hashers"
"github.com/mattermost/mattermost/server/public/shared/request"
"github.com/mattermost/mattermost/server/v8/channels/app/password/hashers"
"github.com/mattermost/mattermost/server/v8/channels/store"
)
+1 -1
View File
@@ -5,7 +5,7 @@
package testlib
import "github.com/mattermost/mattermost/server/v8/channels/app/password/hashers"
import "github.com/mattermost/mattermost/server/public/password/hashers"
func setupFastTestHasher() {
hashers.SetTestHasher(hashers.FastTestHasher())
-1
View File
@@ -18,7 +18,6 @@ require (
github.com/mattermost/gosaml2 v0.10.0
github.com/mattermost/ldap v0.0.0-20231116144001-0f480c025956
github.com/mattermost/logr/v2 v2.0.22
github.com/mattermost/mattermost/server/v8 v8.0.0-20251014075701-833e0125320d
github.com/nicksnyder/go-i18n/v2 v2.6.0
github.com/pborman/uuid v1.2.1
github.com/pkg/errors v0.9.1
-2
View File
@@ -122,8 +122,6 @@ github.com/mattermost/ldap v0.0.0-20231116144001-0f480c025956 h1:Y1Tu/swM31pVwwb
github.com/mattermost/ldap v0.0.0-20231116144001-0f480c025956/go.mod h1:SRl30Lb7/QoYyohYeVBuqYvvmXSZJxZgiV3Zf6VbxjI=
github.com/mattermost/logr/v2 v2.0.22 h1:npFkXlkAWR9J8payh8ftPcCZvLbHSI125mAM5/r/lP4=
github.com/mattermost/logr/v2 v2.0.22/go.mod h1:0sUKpO+XNMZApeumaid7PYaUZPBIydfuWZ0dqixXo+s=
github.com/mattermost/mattermost/server/v8 v8.0.0-20251014075701-833e0125320d h1:etRyN6FNd6fc7BGZ8X+XB2u/5Hb2HNz5/K53YZNvfrs=
github.com/mattermost/mattermost/server/v8 v8.0.0-20251014075701-833e0125320d/go.mod h1:HILhsra+xY4SNEFhuPbobH3I8a0aeXJcTJ6RWPX85nI=
github.com/mattermost/xml-roundtrip-validator v0.1.0 h1:RXbVD2UAl7A7nOTR4u7E3ILa4IbtvKBHw64LDsmu9hU=
github.com/mattermost/xml-roundtrip-validator v0.1.0/go.mod h1:qccnGMcpgwcNaBnxqpJpWWUiPNr5H3O8eDgGV9gT5To=
github.com/mattn/go-colorable v0.1.9/go.mod h1:u6P/XSegPjTcexA+o6vUJrdnUu04hMope9wVRipJSqc=
+1 -1
View File
@@ -19,9 +19,9 @@ import (
"golang.org/x/text/language"
"github.com/mattermost/mattermost/server/public/password/hashers"
"github.com/mattermost/mattermost/server/public/shared/mlog"
"github.com/mattermost/mattermost/server/public/shared/timezones"
"github.com/mattermost/mattermost/server/v8/channels/app/password/hashers"
)
const (
+1 -1
View File
@@ -12,9 +12,9 @@ import (
"github.com/stretchr/testify/assert"
"github.com/stretchr/testify/require"
"github.com/mattermost/mattermost/server/public/password/hashers"
"github.com/mattermost/mattermost/server/public/shared/mlog"
"github.com/mattermost/mattermost/server/public/shared/timezones"
"github.com/mattermost/mattermost/server/v8/channels/app/password/hashers"
)
func TestUserAuditable(t *testing.T) {
@@ -6,7 +6,7 @@ package hashers
import (
"errors"
"github.com/mattermost/mattermost/server/v8/channels/app/password/phcparser"
"github.com/mattermost/mattermost/server/public/password/phcparser"
"golang.org/x/crypto/bcrypt"
)
@@ -8,7 +8,7 @@ import (
"strings"
"testing"
"github.com/mattermost/mattermost/server/v8/channels/app/password/phcparser"
"github.com/mattermost/mattermost/server/public/password/phcparser"
"github.com/stretchr/testify/require"
)
@@ -44,7 +44,7 @@ import (
"fmt"
"strings"
"github.com/mattermost/mattermost/server/v8/channels/app/password/phcparser"
"github.com/mattermost/mattermost/server/public/password/phcparser"
)
// PasswordHasher is a password hasher compliant with the PHC string format:
@@ -6,7 +6,7 @@ package hashers
import (
"testing"
"github.com/mattermost/mattermost/server/v8/channels/app/password/phcparser"
"github.com/mattermost/mattermost/server/public/password/phcparser"
"github.com/stretchr/testify/require"
)
@@ -14,7 +14,7 @@ import (
"strconv"
"strings"
"github.com/mattermost/mattermost/server/v8/channels/app/password/phcparser"
"github.com/mattermost/mattermost/server/public/password/phcparser"
)
const (
@@ -11,7 +11,7 @@ import (
"strings"
"testing"
"github.com/mattermost/mattermost/server/v8/channels/app/password/phcparser"
"github.com/mattermost/mattermost/server/public/password/phcparser"
"github.com/stretchr/testify/require"
)