Address pre-commit errors

The errors being addressed are "new" due to updated configuration.

(I think? It's not entirely clear to me what's up with the typos.)

Author: brianaydemir

config/config.go

@@ -1749,7 +1749,7 @@ func InitServer(ctx context.Context, currentServers server_structs.ServerType) e
 			if !param.Server_EnableUI.GetBool() {
 				return errors.Errorf("%s must be true when %s is set to globus", param.Server_EnableUI.GetName(), param.Origin_StorageType.GetName())
 			}
-			pvd, err := GetOIDCProdiver()
+			pvd, err := GetOIDCProvider()
 			if err != nil || pvd != Globus {
 				log.Info("Server OIDC provider is not Globus. Use Origin.GlobusClientIDFile instead")
 			} else {

config/oidc_metadata.go

@@ -108,7 +108,7 @@ func getMetadataValue(stringParam param.StringParam) (result string, err error)
 		if param.OIDC_Issuer.IsSet() {
 			issuerUrl, _ := url.Parse(param.OIDC_Issuer.GetString())
 			if issuerUrl != nil && issuerUrl.Hostname() == "auth.globus.org" && stringParam.GetName() == param.OIDC_DeviceAuthEndpoint.GetName() {
-				log.Warning("You are using Globus as the auth privider. Although it does not support OAuth device flow used by Pelican registry, you may use it for other Pelican servers. OIDC.DeviceAuthEndpoint is set to https://auth.globus.org/")
+				log.Warning("You are using Globus as the auth provider. Although it does not support OAuth device flow used by Pelican registry, you may use it for other Pelican servers. OIDC.DeviceAuthEndpoint is set to https://auth.globus.org/")
 				result = "https://auth.globus.org/"
 				return
 			}
@@ -130,7 +130,7 @@ func getMetadataValue(stringParam param.StringParam) (result string, err error)
 }
 
 // Get from the config parameters the OIDC provider
-func GetOIDCProdiver() (pvd OIDCProvider, err error) {
+func GetOIDCProvider() (pvd OIDCProvider, err error) {
 	authURLStr := param.OIDC_AuthorizationEndpoint.GetString()
 	if authURLStr == "" {
 		authURLStr = param.OIDC_Issuer.GetString()

config/oidc_metadata_test.go

@@ -34,7 +34,7 @@ func TestGetOIDCProvider(t *testing.T) {
 	})
 	t.Run("empty-endpoints-gives-error", func(t *testing.T) {
 		ResetConfig()
-		get, err := GetOIDCProdiver()
+		get, err := GetOIDCProvider()
 		require.Error(t, err)
 		assert.Contains(t, err.Error(), "nothing set for config parameter OIDC.IssuerUrl or OIDC.AuthorizationEndpoint")
 		assert.Empty(t, get)
@@ -43,45 +43,45 @@ func TestGetOIDCProvider(t *testing.T) {
 	t.Run("auth-endpoint-gives-correct-result", func(t *testing.T) {
 		ResetConfig()
 		require.NoError(t, param.Set(param.OIDC_AuthorizationEndpoint.GetName(), "https://example.com/authorization"))
-		get, err := GetOIDCProdiver()
+		get, err := GetOIDCProvider()
 		require.NoError(t, err)
 		assert.Equal(t, UnknownProvider, get)
 
 		// CILogon
 		require.NoError(t, param.Set(param.OIDC_AuthorizationEndpoint.GetName(), "https://cilogon.org/api/v1.0/authorization"))
-		get, err = GetOIDCProdiver()
+		get, err = GetOIDCProvider()
 		require.NoError(t, err)
 		assert.Equal(t, CILogon, get)
 
 		// Globus
 		require.NoError(t, param.Set(param.OIDC_AuthorizationEndpoint.GetName(), "https://auth.globus.org/api/v1.0/authorization"))
-		get, err = GetOIDCProdiver()
+		get, err = GetOIDCProvider()
 		require.NoError(t, err)
 		assert.Equal(t, Globus, get)
 	})
 
 	t.Run("issuer-endpoint-gives-correct-result", func(t *testing.T) {
 		ResetConfig()
 		require.NoError(t, param.Set(param.OIDC_Issuer.GetName(), "https://example.com"))
-		get, err := GetOIDCProdiver()
+		get, err := GetOIDCProvider()
 		require.NoError(t, err)
 		assert.Equal(t, UnknownProvider, get)
 
 		// CILogon
 		require.NoError(t, param.Set(param.OIDC_AuthorizationEndpoint.GetName(), "https://cilogon.org"))
-		get, err = GetOIDCProdiver()
+		get, err = GetOIDCProvider()
 		require.NoError(t, err)
 		assert.Equal(t, CILogon, get)
 
 		// CILogon no protocol
 		require.NoError(t, param.Set(param.OIDC_AuthorizationEndpoint.GetName(), "cilogon.org"))
-		get, err = GetOIDCProdiver()
+		get, err = GetOIDCProvider()
 		require.NoError(t, err)
 		assert.Equal(t, CILogon, get)
 
 		// Globus no protocol
 		require.NoError(t, param.Set(param.OIDC_AuthorizationEndpoint.GetName(), "auth.globus.org"))
-		get, err = GetOIDCProdiver()
+		get, err = GetOIDCProvider()
 		require.NoError(t, err)
 		assert.Equal(t, Globus, get)
 	})

docs/app/getting-help/faqs/page.mdx

@@ -20,7 +20,7 @@ and we frequently refer to said components as "Pelican _____" in our documentati
 
 ### How is the Pelican Platform related to the HTCondor Software Suite (HTCSS)?
 
-Both the Pelican Platform and [HTCSS](https://htcondor.org/) are open-source software techonologies whose development is led by the [Center for High Throughput Computing (CHTC)](https://chtc.cs.wisc.edu) at the University of Wisconsin-Madison,
+Both the Pelican Platform and [HTCSS](https://htcondor.org/) are open-source software technologies whose development is led by the [Center for High Throughput Computing (CHTC)](https://chtc.cs.wisc.edu) at the University of Wisconsin-Madison,
 whose mission is "to bring the power of High Throughput Computing to all fields of research, and to allow the future of HTC to be shaped by insight from all fields."
 While the funding for developing these technologies comes from different sources, there is a lot of overlap of the staff that are involved in the development of these technologies.
 

metrics/xrootd_metrics.go

@@ -468,7 +468,7 @@ var (
 	}, []string{"path", "ap", "dn", "role", "org", "proj", "network"})
 
 	TransferReadvSegsTotal = promauto.NewCounterVec(prometheus.CounterOpts{
-		Name: "xrootd_transfer_readv_segments_toal",
+		Name: "xrootd_transfer_readv_segments_total",
 		Help: "Number of segments in readv operations",
 	}, []string{"path", "ap", "dn", "role", "org", "proj", "network"})
 
@@ -866,7 +866,7 @@ var (
 		Buckets: TimeHistogramBuckets,
 	})
 
-	CPUUtilzation = promauto.NewGauge(prometheus.GaugeOpts{
+	CPUUtilization = promauto.NewGauge(prometheus.GaugeOpts{
 		Name: "xrootd_cpu_utilization",
 		Help: "CPU utilization of the XRootD server",
 	})
@@ -1964,8 +1964,8 @@ func HandleSummaryPacket(packet []byte) error {
 
 				if wallDelta > 0 {
 					// represents the average of cores utilized during the interval
-					utlizationRatio := cpuDelta / wallDelta
-					CPUUtilzation.Set(utlizationRatio)
+					utilizationRatio := cpuDelta / wallDelta
+					CPUUtilization.Set(utilizationRatio)
 				}
 			}
 

registry/registry_db_test.go

@@ -158,7 +158,7 @@ func mockNamespace(prefix, pubkey, identity string, adminMetadata server_structs
 }
 
 // Some genertic mock data function to be shared with other test
-// functinos in this package. Please treat them as "constants"
+// functions in this package. Please treat them as "constants"
 var (
 	mockNssWithNamespaces []server_structs.Registration = []server_structs.Registration{
 		mockNamespace("/test1", "pubkey1", "", server_structs.AdminMetadata{Status: server_structs.RegApproved}),

registry/registry_validation.go

@@ -202,7 +202,7 @@ func validateInstitution(instID string) (bool, error) {
 		} else {
 			insts, err := getCachedOptions(instUrl, instUrlTTL)
 			if err != nil {
-				return false, errors.Wrap(err, "Error fetching instituions from TTL cache")
+				return false, errors.Wrap(err, "Error fetching institutions from TTL cache")
 			}
 			for _, availableInst := range insts {
 				// We required full equality, as we expect the value is from the institution API

server_structs/director.go

@@ -555,7 +555,7 @@ func (ad *ServerBaseAd) Initialize(name string) {
 	ad.StartTime = startTime
 	ad.GenerationID = generationID.Add(1) + 1
 	adLifetime := param.Server_AdLifetime.GetDuration()
-	// Not all unit tests initializae the config, meaning adLifetime
+	// Not all unit tests initialize the config, meaning adLifetime
 	// may be 0.
 	if adLifetime == 0 {
 		adLifetime = 15 * time.Minute

swagger/pelican-swagger.yaml

@@ -275,7 +275,7 @@ definitions:
         type: string
         description:
           The public JWK from the origin that wants to register the namespace.
-          It should be a marshaled (stringfied) JSON that contains either one JWK or a JWKS
+          It should be a marshaled (stringified) JSON that contains either one JWK or a JWKS
       admin_metadata:
         $ref: "#/definitions/AdminMetadata"
       custom_fields:
@@ -359,7 +359,7 @@ definitions:
         type: string
         description:
           The public JWK from the origin that wants to register the namespace.
-          It should be a marshaled (stringfied) JSON that contains either one JWK or a JWKS
+          It should be a marshaled (stringified) JSON that contains either one JWK or a JWKS
       admin_metadata:
         $ref: "#/definitions/AdminMetadataForRegistration"
       custom_fields:

web_ui/frontend/dev/image/Dockerfile

@@ -10,15 +10,13 @@ RUN apt-get update && apt-get install -y cron
 
 # Update the entrypoint
 COPY ./entrypoint.sh /opt/bin/entrypoint.sh
-RUN chmod +x /opt/bin/entrypoint.sh
 ENTRYPOINT ["/opt/bin/entrypoint.sh"]
 
 # Copy the NGINX configuration file
 COPY ./nginx.conf /etc/nginx/templates/nginx.conf.template
 
 # Copy the login script
 COPY ./login.sh /opt/bin/login.sh
-RUN chmod +x /opt/bin/login.sh
 
 # Copy the crontab file and set up cron job
 COPY ./crontab /etc/cron.d/login_cron