Skip to content

1095- ServiceNow backend endpoint code changes#197

Open
dineshS96 wants to merge 15 commits intodevfrom
1095-ServiceNowBackEndChanges
Open

1095- ServiceNow backend endpoint code changes#197
dineshS96 wants to merge 15 commits intodevfrom
1095-ServiceNowBackEndChanges

Conversation

@dineshS96
Copy link

@dineshS96 dineshS96 commented Sep 26, 2025

Description

Service Now Connector backend code changes for new endpoint

Changes Made

UserResource.java

How to Test

1.Login in SEMOSS application using native login
2.Need to pass required commands in respective Reactor in terminal

@dineshS96 dineshS96 requested a review from a team as a code owner September 26, 2025 12:52
@anurag91jain anurag91jain marked this pull request as draft September 26, 2025 13:36
@anurag91jain anurag91jain linked an issue Sep 26, 2025 that may be closed by this pull request
1 task
String instanceUrl = null;
String clientId = null;
String clientSecret = null;
String redirectUri = "http://localhost:9090/Monolith/api/auth/login2/servicenow";

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can we get this from social.property file or call hosted url +/Monolith/api/auth/login2/servicenow

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Need to check

private String getServiceNowRedirectDb(HttpServletRequest request, @Context HttpServletResponse response) throws UnsupportedEncodingException, IOException {
String instanceUrl = null;
String clientId = null;
String redirectUri = "http://localhost:9090/Monolith/api/auth/login2/servicenow";

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We can't hardcode the uri.

@snehakumari369 snehakumari369 marked this pull request as ready for review February 13, 2026 14:25
@snehakumari369 snehakumari369 requested review from Pragya7011 and snehakumari369 and removed request for snehakumari369 February 16, 2026 12:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants