mirror of
				https://github.com/go-gitea/gitea.git
				synced 2025-10-26 12:27:06 +00:00 
			
		
		
		
	Use crypto/sha256 (#29386)
				
					
				
			Go 1.21 improved the performance of `crypto/sha256`. It's now similar to `minio/sha256-simd`, so we should just use the standard libs. https://go.dev/doc/go1.21#crypto/sha256 https://go-review.googlesource.com/c/go/+/408795 https://github.com/multiformats/go-multihash/pull/173
This commit is contained in:
		
							
								
								
									
										2
									
								
								go.mod
									
									
									
									
									
								
							
							
						
						
									
										2
									
								
								go.mod
									
									
									
									
									
								
							| @@ -78,7 +78,6 @@ require ( | |||||||
| 	github.com/mholt/archiver/v3 v3.5.1 | 	github.com/mholt/archiver/v3 v3.5.1 | ||||||
| 	github.com/microcosm-cc/bluemonday v1.0.26 | 	github.com/microcosm-cc/bluemonday v1.0.26 | ||||||
| 	github.com/minio/minio-go/v7 v7.0.66 | 	github.com/minio/minio-go/v7 v7.0.66 | ||||||
| 	github.com/minio/sha256-simd v1.0.1 |  | ||||||
| 	github.com/msteinert/pam v1.2.0 | 	github.com/msteinert/pam v1.2.0 | ||||||
| 	github.com/nektos/act v0.2.52 | 	github.com/nektos/act v0.2.52 | ||||||
| 	github.com/niklasfasching/go-org v1.7.0 | 	github.com/niklasfasching/go-org v1.7.0 | ||||||
| @@ -230,6 +229,7 @@ require ( | |||||||
| 	github.com/mholt/acmez v1.2.0 // indirect | 	github.com/mholt/acmez v1.2.0 // indirect | ||||||
| 	github.com/miekg/dns v1.1.58 // indirect | 	github.com/miekg/dns v1.1.58 // indirect | ||||||
| 	github.com/minio/md5-simd v1.1.2 // indirect | 	github.com/minio/md5-simd v1.1.2 // indirect | ||||||
|  | 	github.com/minio/sha256-simd v1.0.1 // indirect | ||||||
| 	github.com/mitchellh/copystructure v1.2.0 // indirect | 	github.com/mitchellh/copystructure v1.2.0 // indirect | ||||||
| 	github.com/mitchellh/mapstructure v1.5.0 // indirect | 	github.com/mitchellh/mapstructure v1.5.0 // indirect | ||||||
| 	github.com/mitchellh/reflectwalk v1.0.2 // indirect | 	github.com/mitchellh/reflectwalk v1.0.2 // indirect | ||||||
|   | |||||||
| @@ -6,6 +6,7 @@ package auth | |||||||
| import ( | import ( | ||||||
| 	"context" | 	"context" | ||||||
| 	"crypto/md5" | 	"crypto/md5" | ||||||
|  | 	"crypto/sha256" | ||||||
| 	"crypto/subtle" | 	"crypto/subtle" | ||||||
| 	"encoding/base32" | 	"encoding/base32" | ||||||
| 	"encoding/base64" | 	"encoding/base64" | ||||||
| @@ -18,7 +19,6 @@ import ( | |||||||
| 	"code.gitea.io/gitea/modules/timeutil" | 	"code.gitea.io/gitea/modules/timeutil" | ||||||
| 	"code.gitea.io/gitea/modules/util" | 	"code.gitea.io/gitea/modules/util" | ||||||
|  |  | ||||||
| 	"github.com/minio/sha256-simd" |  | ||||||
| 	"github.com/pquerna/otp/totp" | 	"github.com/pquerna/otp/totp" | ||||||
| 	"golang.org/x/crypto/pbkdf2" | 	"golang.org/x/crypto/pbkdf2" | ||||||
| ) | ) | ||||||
|   | |||||||
| @@ -4,9 +4,9 @@ | |||||||
| package base | package base | ||||||
|  |  | ||||||
| import ( | import ( | ||||||
|  | 	"crypto/sha256" | ||||||
| 	"encoding/hex" | 	"encoding/hex" | ||||||
|  |  | ||||||
| 	"github.com/minio/sha256-simd" |  | ||||||
| 	"golang.org/x/crypto/pbkdf2" | 	"golang.org/x/crypto/pbkdf2" | ||||||
| ) | ) | ||||||
|  |  | ||||||
|   | |||||||
| @@ -4,9 +4,9 @@ | |||||||
| package v1_14 //nolint | package v1_14 //nolint | ||||||
|  |  | ||||||
| import ( | import ( | ||||||
|  | 	"crypto/sha256" | ||||||
| 	"encoding/hex" | 	"encoding/hex" | ||||||
|  |  | ||||||
| 	"github.com/minio/sha256-simd" |  | ||||||
| 	"golang.org/x/crypto/argon2" | 	"golang.org/x/crypto/argon2" | ||||||
| 	"golang.org/x/crypto/bcrypt" | 	"golang.org/x/crypto/bcrypt" | ||||||
| 	"golang.org/x/crypto/pbkdf2" | 	"golang.org/x/crypto/pbkdf2" | ||||||
|   | |||||||
| @@ -4,12 +4,12 @@ | |||||||
| package hash | package hash | ||||||
|  |  | ||||||
| import ( | import ( | ||||||
|  | 	"crypto/sha256" | ||||||
| 	"encoding/hex" | 	"encoding/hex" | ||||||
| 	"strings" | 	"strings" | ||||||
|  |  | ||||||
| 	"code.gitea.io/gitea/modules/log" | 	"code.gitea.io/gitea/modules/log" | ||||||
|  |  | ||||||
| 	"github.com/minio/sha256-simd" |  | ||||||
| 	"golang.org/x/crypto/pbkdf2" | 	"golang.org/x/crypto/pbkdf2" | ||||||
| ) | ) | ||||||
|  |  | ||||||
|   | |||||||
| @@ -4,10 +4,9 @@ | |||||||
| package avatar | package avatar | ||||||
|  |  | ||||||
| import ( | import ( | ||||||
|  | 	"crypto/sha256" | ||||||
| 	"encoding/hex" | 	"encoding/hex" | ||||||
| 	"strconv" | 	"strconv" | ||||||
|  |  | ||||||
| 	"github.com/minio/sha256-simd" |  | ||||||
| ) | ) | ||||||
|  |  | ||||||
| // HashAvatar will generate a unique string, which ensures that when there's a | // HashAvatar will generate a unique string, which ensures that when there's a | ||||||
|   | |||||||
| @@ -7,11 +7,10 @@ | |||||||
| package identicon | package identicon | ||||||
|  |  | ||||||
| import ( | import ( | ||||||
|  | 	"crypto/sha256" | ||||||
| 	"fmt" | 	"fmt" | ||||||
| 	"image" | 	"image" | ||||||
| 	"image/color" | 	"image/color" | ||||||
|  |  | ||||||
| 	"github.com/minio/sha256-simd" |  | ||||||
| ) | ) | ||||||
|  |  | ||||||
| const minImageSize = 16 | const minImageSize = 16 | ||||||
|   | |||||||
| @@ -5,6 +5,7 @@ package base | |||||||
|  |  | ||||||
| import ( | import ( | ||||||
| 	"crypto/sha1" | 	"crypto/sha1" | ||||||
|  | 	"crypto/sha256" | ||||||
| 	"encoding/base64" | 	"encoding/base64" | ||||||
| 	"encoding/hex" | 	"encoding/hex" | ||||||
| 	"errors" | 	"errors" | ||||||
| @@ -22,7 +23,6 @@ import ( | |||||||
| 	"code.gitea.io/gitea/modules/setting" | 	"code.gitea.io/gitea/modules/setting" | ||||||
|  |  | ||||||
| 	"github.com/dustin/go-humanize" | 	"github.com/dustin/go-humanize" | ||||||
| 	"github.com/minio/sha256-simd" |  | ||||||
| ) | ) | ||||||
|  |  | ||||||
| // EncodeSha1 string to sha1 hex value. | // EncodeSha1 string to sha1 hex value. | ||||||
|   | |||||||
| @@ -4,12 +4,11 @@ | |||||||
| package git | package git | ||||||
|  |  | ||||||
| import ( | import ( | ||||||
|  | 	"crypto/sha256" | ||||||
| 	"fmt" | 	"fmt" | ||||||
|  |  | ||||||
| 	"code.gitea.io/gitea/modules/log" | 	"code.gitea.io/gitea/modules/log" | ||||||
| 	"code.gitea.io/gitea/modules/setting" | 	"code.gitea.io/gitea/modules/setting" | ||||||
|  |  | ||||||
| 	"github.com/minio/sha256-simd" |  | ||||||
| ) | ) | ||||||
|  |  | ||||||
| // Cache represents a caching interface | // Cache represents a caching interface | ||||||
|   | |||||||
| @@ -4,6 +4,7 @@ | |||||||
| package lfs | package lfs | ||||||
|  |  | ||||||
| import ( | import ( | ||||||
|  | 	"crypto/sha256" | ||||||
| 	"encoding/hex" | 	"encoding/hex" | ||||||
| 	"errors" | 	"errors" | ||||||
| 	"hash" | 	"hash" | ||||||
| @@ -12,8 +13,6 @@ import ( | |||||||
|  |  | ||||||
| 	"code.gitea.io/gitea/modules/log" | 	"code.gitea.io/gitea/modules/log" | ||||||
| 	"code.gitea.io/gitea/modules/storage" | 	"code.gitea.io/gitea/modules/storage" | ||||||
|  |  | ||||||
| 	"github.com/minio/sha256-simd" |  | ||||||
| ) | ) | ||||||
|  |  | ||||||
| var ( | var ( | ||||||
|   | |||||||
| @@ -4,6 +4,7 @@ | |||||||
| package lfs | package lfs | ||||||
|  |  | ||||||
| import ( | import ( | ||||||
|  | 	"crypto/sha256" | ||||||
| 	"encoding/hex" | 	"encoding/hex" | ||||||
| 	"errors" | 	"errors" | ||||||
| 	"fmt" | 	"fmt" | ||||||
| @@ -12,8 +13,6 @@ import ( | |||||||
| 	"regexp" | 	"regexp" | ||||||
| 	"strconv" | 	"strconv" | ||||||
| 	"strings" | 	"strings" | ||||||
|  |  | ||||||
| 	"github.com/minio/sha256-simd" |  | ||||||
| ) | ) | ||||||
|  |  | ||||||
| const ( | const ( | ||||||
|   | |||||||
| @@ -7,13 +7,12 @@ import ( | |||||||
| 	"crypto/aes" | 	"crypto/aes" | ||||||
| 	"crypto/cipher" | 	"crypto/cipher" | ||||||
| 	"crypto/rand" | 	"crypto/rand" | ||||||
|  | 	"crypto/sha256" | ||||||
| 	"encoding/base64" | 	"encoding/base64" | ||||||
| 	"encoding/hex" | 	"encoding/hex" | ||||||
| 	"errors" | 	"errors" | ||||||
| 	"fmt" | 	"fmt" | ||||||
| 	"io" | 	"io" | ||||||
|  |  | ||||||
| 	"github.com/minio/sha256-simd" |  | ||||||
| ) | ) | ||||||
|  |  | ||||||
| // AesEncrypt encrypts text and given key with AES. | // AesEncrypt encrypts text and given key with AES. | ||||||
|   | |||||||
| @@ -7,10 +7,9 @@ import ( | |||||||
| 	"crypto" | 	"crypto" | ||||||
| 	"crypto/rand" | 	"crypto/rand" | ||||||
| 	"crypto/rsa" | 	"crypto/rsa" | ||||||
|  | 	"crypto/sha256" | ||||||
| 	"crypto/x509" | 	"crypto/x509" | ||||||
| 	"encoding/pem" | 	"encoding/pem" | ||||||
|  |  | ||||||
| 	"github.com/minio/sha256-simd" |  | ||||||
| ) | ) | ||||||
|  |  | ||||||
| // GenerateKeyPair generates a public and private keypair | // GenerateKeyPair generates a public and private keypair | ||||||
|   | |||||||
| @@ -7,12 +7,12 @@ import ( | |||||||
| 	"crypto" | 	"crypto" | ||||||
| 	"crypto/rand" | 	"crypto/rand" | ||||||
| 	"crypto/rsa" | 	"crypto/rsa" | ||||||
|  | 	"crypto/sha256" | ||||||
| 	"crypto/x509" | 	"crypto/x509" | ||||||
| 	"encoding/pem" | 	"encoding/pem" | ||||||
| 	"regexp" | 	"regexp" | ||||||
| 	"testing" | 	"testing" | ||||||
|  |  | ||||||
| 	"github.com/minio/sha256-simd" |  | ||||||
| 	"github.com/stretchr/testify/assert" | 	"github.com/stretchr/testify/assert" | ||||||
| ) | ) | ||||||
|  |  | ||||||
|   | |||||||
| @@ -8,6 +8,7 @@ import ( | |||||||
| 	"crypto" | 	"crypto" | ||||||
| 	"crypto/rsa" | 	"crypto/rsa" | ||||||
| 	"crypto/sha1" | 	"crypto/sha1" | ||||||
|  | 	"crypto/sha256" | ||||||
| 	"crypto/x509" | 	"crypto/x509" | ||||||
| 	"encoding/base64" | 	"encoding/base64" | ||||||
| 	"encoding/pem" | 	"encoding/pem" | ||||||
| @@ -26,8 +27,6 @@ import ( | |||||||
| 	chef_module "code.gitea.io/gitea/modules/packages/chef" | 	chef_module "code.gitea.io/gitea/modules/packages/chef" | ||||||
| 	"code.gitea.io/gitea/modules/util" | 	"code.gitea.io/gitea/modules/util" | ||||||
| 	"code.gitea.io/gitea/services/auth" | 	"code.gitea.io/gitea/services/auth" | ||||||
|  |  | ||||||
| 	"github.com/minio/sha256-simd" |  | ||||||
| ) | ) | ||||||
|  |  | ||||||
| const ( | const ( | ||||||
|   | |||||||
| @@ -6,6 +6,7 @@ package maven | |||||||
| import ( | import ( | ||||||
| 	"crypto/md5" | 	"crypto/md5" | ||||||
| 	"crypto/sha1" | 	"crypto/sha1" | ||||||
|  | 	"crypto/sha256" | ||||||
| 	"crypto/sha512" | 	"crypto/sha512" | ||||||
| 	"encoding/hex" | 	"encoding/hex" | ||||||
| 	"encoding/xml" | 	"encoding/xml" | ||||||
| @@ -26,8 +27,6 @@ import ( | |||||||
| 	maven_module "code.gitea.io/gitea/modules/packages/maven" | 	maven_module "code.gitea.io/gitea/modules/packages/maven" | ||||||
| 	"code.gitea.io/gitea/routers/api/packages/helper" | 	"code.gitea.io/gitea/routers/api/packages/helper" | ||||||
| 	packages_service "code.gitea.io/gitea/services/packages" | 	packages_service "code.gitea.io/gitea/services/packages" | ||||||
|  |  | ||||||
| 	"github.com/minio/sha256-simd" |  | ||||||
| ) | ) | ||||||
|  |  | ||||||
| const ( | const ( | ||||||
|   | |||||||
| @@ -5,6 +5,7 @@ package lfs | |||||||
|  |  | ||||||
| import ( | import ( | ||||||
| 	stdCtx "context" | 	stdCtx "context" | ||||||
|  | 	"crypto/sha256" | ||||||
| 	"encoding/base64" | 	"encoding/base64" | ||||||
| 	"encoding/hex" | 	"encoding/hex" | ||||||
| 	"errors" | 	"errors" | ||||||
| @@ -33,7 +34,6 @@ import ( | |||||||
| 	"code.gitea.io/gitea/modules/storage" | 	"code.gitea.io/gitea/modules/storage" | ||||||
|  |  | ||||||
| 	"github.com/golang-jwt/jwt/v5" | 	"github.com/golang-jwt/jwt/v5" | ||||||
| 	"github.com/minio/sha256-simd" |  | ||||||
| ) | ) | ||||||
|  |  | ||||||
| // requestContext contain variables from the HTTP request. | // requestContext contain variables from the HTTP request. | ||||||
|   | |||||||
| @@ -6,14 +6,13 @@ package token | |||||||
| import ( | import ( | ||||||
| 	"context" | 	"context" | ||||||
| 	crypto_hmac "crypto/hmac" | 	crypto_hmac "crypto/hmac" | ||||||
|  | 	"crypto/sha256" | ||||||
| 	"encoding/base32" | 	"encoding/base32" | ||||||
| 	"fmt" | 	"fmt" | ||||||
| 	"time" | 	"time" | ||||||
|  |  | ||||||
| 	user_model "code.gitea.io/gitea/models/user" | 	user_model "code.gitea.io/gitea/models/user" | ||||||
| 	"code.gitea.io/gitea/modules/util" | 	"code.gitea.io/gitea/modules/util" | ||||||
|  |  | ||||||
| 	"github.com/minio/sha256-simd" |  | ||||||
| ) | ) | ||||||
|  |  | ||||||
| // A token is a verifiable container describing an action. | // A token is a verifiable container describing an action. | ||||||
|   | |||||||
| @@ -7,6 +7,7 @@ import ( | |||||||
| 	"context" | 	"context" | ||||||
| 	"crypto/hmac" | 	"crypto/hmac" | ||||||
| 	"crypto/sha1" | 	"crypto/sha1" | ||||||
|  | 	"crypto/sha256" | ||||||
| 	"crypto/tls" | 	"crypto/tls" | ||||||
| 	"encoding/hex" | 	"encoding/hex" | ||||||
| 	"fmt" | 	"fmt" | ||||||
| @@ -29,7 +30,6 @@ import ( | |||||||
| 	webhook_module "code.gitea.io/gitea/modules/webhook" | 	webhook_module "code.gitea.io/gitea/modules/webhook" | ||||||
|  |  | ||||||
| 	"github.com/gobwas/glob" | 	"github.com/gobwas/glob" | ||||||
| 	"github.com/minio/sha256-simd" |  | ||||||
| ) | ) | ||||||
|  |  | ||||||
| // Deliver deliver hook task | // Deliver deliver hook task | ||||||
|   | |||||||
| @@ -11,6 +11,7 @@ import ( | |||||||
| 	"crypto/rand" | 	"crypto/rand" | ||||||
| 	"crypto/rsa" | 	"crypto/rsa" | ||||||
| 	"crypto/sha1" | 	"crypto/sha1" | ||||||
|  | 	"crypto/sha256" | ||||||
| 	"crypto/x509" | 	"crypto/x509" | ||||||
| 	"encoding/base64" | 	"encoding/base64" | ||||||
| 	"encoding/pem" | 	"encoding/pem" | ||||||
| @@ -33,7 +34,6 @@ import ( | |||||||
| 	chef_router "code.gitea.io/gitea/routers/api/packages/chef" | 	chef_router "code.gitea.io/gitea/routers/api/packages/chef" | ||||||
| 	"code.gitea.io/gitea/tests" | 	"code.gitea.io/gitea/tests" | ||||||
|  |  | ||||||
| 	"github.com/minio/sha256-simd" |  | ||||||
| 	"github.com/stretchr/testify/assert" | 	"github.com/stretchr/testify/assert" | ||||||
| ) | ) | ||||||
|  |  | ||||||
|   | |||||||
| @@ -5,6 +5,7 @@ package integration | |||||||
|  |  | ||||||
| import ( | import ( | ||||||
| 	"bytes" | 	"bytes" | ||||||
|  | 	"crypto/sha256" | ||||||
| 	"encoding/base64" | 	"encoding/base64" | ||||||
| 	"fmt" | 	"fmt" | ||||||
| 	"net/http" | 	"net/http" | ||||||
| @@ -24,7 +25,6 @@ import ( | |||||||
| 	"code.gitea.io/gitea/modules/test" | 	"code.gitea.io/gitea/modules/test" | ||||||
| 	"code.gitea.io/gitea/tests" | 	"code.gitea.io/gitea/tests" | ||||||
|  |  | ||||||
| 	"github.com/minio/sha256-simd" |  | ||||||
| 	oci "github.com/opencontainers/image-spec/specs-go/v1" | 	oci "github.com/opencontainers/image-spec/specs-go/v1" | ||||||
| 	"github.com/stretchr/testify/assert" | 	"github.com/stretchr/testify/assert" | ||||||
| ) | ) | ||||||
|   | |||||||
| @@ -5,6 +5,7 @@ package integration | |||||||
|  |  | ||||||
| import ( | import ( | ||||||
| 	"bytes" | 	"bytes" | ||||||
|  | 	"crypto/sha256" | ||||||
| 	"fmt" | 	"fmt" | ||||||
| 	"net/http" | 	"net/http" | ||||||
| 	"strings" | 	"strings" | ||||||
| @@ -24,7 +25,6 @@ import ( | |||||||
| 	packages_cleanup_service "code.gitea.io/gitea/services/packages/cleanup" | 	packages_cleanup_service "code.gitea.io/gitea/services/packages/cleanup" | ||||||
| 	"code.gitea.io/gitea/tests" | 	"code.gitea.io/gitea/tests" | ||||||
|  |  | ||||||
| 	"github.com/minio/sha256-simd" |  | ||||||
| 	"github.com/stretchr/testify/assert" | 	"github.com/stretchr/testify/assert" | ||||||
| ) | ) | ||||||
|  |  | ||||||
|   | |||||||
		Reference in New Issue
	
	Block a user
	 KN4CK3R
					KN4CK3R