Skip to content

Commit

Permalink
Fix Personal Access Token login method to Github Enterprise Server mo…
Browse files Browse the repository at this point in the history
…difying getUserData (#524)

* Fix Personal Access Token login method to Github Enterprise Server

* Apply prettier

Co-authored-by: Manos Konstantinidis <[email protected]>
  • Loading branch information
yyar and manosim authored Nov 16, 2021
1 parent f446794 commit f8a6ac2
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 4 deletions.
4 changes: 2 additions & 2 deletions src/context/App.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -113,8 +113,8 @@ export const AppProvider = ({ children }: { children: React.ReactNode }) => {
const login = useCallback(async () => {
const { authCode } = await authGitHub();
const { token } = await getToken(authCode);
const user = await getUserData(token);
const hostname = Constants.DEFAULT_AUTH_OPTIONS.hostname;
const user = await getUserData(token, hostname);
const updatedAccounts = addAccount(accounts, token, hostname, user);
setAccounts(updatedAccounts);
saveState(updatedAccounts, settings);
Expand All @@ -138,7 +138,7 @@ export const AppProvider = ({ children }: { children: React.ReactNode }) => {
'HEAD',
token
);
const user = await getUserData(token);
const user = await getUserData(token, hostname);
const updatedAccounts = addAccount(accounts, token, hostname, user);
setAccounts(updatedAccounts);
saveState(updatedAccounts, settings);
Expand Down
7 changes: 5 additions & 2 deletions src/utils/auth.ts
Original file line number Diff line number Diff line change
Expand Up @@ -73,9 +73,12 @@ export const authGitHub = (
});
};

export const getUserData = async (token: string): Promise<User> => {
export const getUserData = async (
token: string,
hostname: string
): Promise<User> => {
const response = await apiRequestAuth(
`https://api.${Constants.DEFAULT_AUTH_OPTIONS.hostname}/user`,
`https://api.${hostname}/user`,
'GET',
token
);
Expand Down

0 comments on commit f8a6ac2

Please sign in to comment.