Skip to content

Commit c9a6ee1

Browse files
committed
convert whitelist to exempt list
1 parent ec82896 commit c9a6ee1

File tree

1 file changed

+12
-32
lines changed

1 file changed

+12
-32
lines changed

internal/router/auth.go

Lines changed: 12 additions & 32 deletions
Original file line numberDiff line numberDiff line change
@@ -181,40 +181,20 @@ func (r *RouterServer) AuthHandler(ctx *context.Context, h http.Handler) http.Ha
181181

182182
// CustomAuth
183183
if isBasicAuth {
184-
allowedUsers := []string{
185-
"upi-offering-service-payments.de-apps@razorpay.com",
186-
"reporting-service-platform-r1cxp.de-apps@razorpay.com",
187-
"cross-border-import-service-payments.de-apps@razorpay.com",
188-
"1cc-shipping-service-payments.de-apps@razorpay.com",
189-
"catalyst-service-payments.de-apps@razorpay.com",
190-
"catalyst-service-payments.de-apps%40razorpay.com",
191-
"cross-border-payments-service-payments.de-apps@razorpay.com",
192-
"cross-border-payments-service-payments.de-apps%40razorpay.com",
193-
"cmma-service-payments.de-apps@razorpay.com",
194-
"cmma-service-payments.de-apps%40razorpay.com",
195-
"merchants-risk-payments.de-apps@razorpay.com",
196-
"merchants-risk-payments.de-apps%40razorpay.com",
197-
"charge-collections-service-platform-r1cxp.de-apps@razorpay.com",
198-
"charge-collections-service-platform-r1cxp.de-apps%40razorpay.com",
199-
"payments-bank-transfers-service-payments.de-apps@razorpay.com",
200-
"payments-bank-transfers-service-payments.de-apps%40razorpay.com",
201-
"ade-service-security.de-apps@razorpay.com",
202-
"ade-service-security.de-apps%40razorpay.com",
203-
"merchant-experience-service-platforms.de-apps@razorpay.com",
204-
"merchant-experience-service-platforms.de-apps%40razorpay.com",
205-
"insights-service-data-org.de-apps@razorpay.com",
206-
"insights-service-data-org.de-apps%40razorpay.com",
207-
"governor-service-payments.de-apps@razorpay.com",
208-
"governor-service-payments.de-apps%40razorpay.com",
209-
"shield-service-payments.de-apps@razorpay.com",
210-
"shield-service-payments.de-apps%40razorpay.com",
184+
exemptedUsers := []string{
185+
"capital-scorecard",
186+
"care",
187+
"cyber-helpdesk",
188+
"datum",
189+
"disputes",
190+
"magic-checkout",
191+
"partnerships",
192+
"prod_api",
211193
"api-service-payments.de-apps@razorpay.com",
212-
"api-service-payments.de-apps%40razorpay.com",
213-
"pgos-service-payments.de-apps@razorpay.com",
214-
"pgos-service-payments.de-apps%40razorpay.com",
194+
"settlements",
215195
}
216196

217-
if utils.SliceContains(allowedUsers, username) {
197+
if !utils.SliceContains(exemptedUsers, username) {
218198
if u := trinoheaders.Get(trinoheaders.User, req); u != username {
219199
errorMsg := fmt.Sprintf("Username from basicauth - %s does not match with User principal - %s", username, u)
220200
provider.Logger(*ctx).Debug(errorMsg)
@@ -226,7 +206,7 @@ func (r *RouterServer) AuthHandler(ctx *context.Context, h http.Handler) http.Ha
226206
req.Header.Del("Authorization")
227207
isAuthenticated, err := r.authService.Authenticate(ctx, username, password)
228208
if err != nil {
229-
errorMsg := fmt.Sprintf("Unable to Authenticate users. Getting error - %s", err)
209+
errorMsg := fmt.Sprintf("Unable to Authenticate user: %s. Getting error - %s", username, err)
230210
provider.Logger(*ctx).Error(errorMsg)
231211
http.Error(w, "Unable to Authenticate the user", http.StatusNotFound)
232212
return

0 commit comments

Comments
 (0)