Skip to content
This repository has been archived by the owner on Dec 13, 2023. It is now read-only.

Commit

Permalink
[Hot-fix] fix: remove redundant slash that cause proxy fail (#3873)
Browse files Browse the repository at this point in the history
  • Loading branch information
nhandt2021 authored Nov 30, 2023
1 parent 0f187bf commit 41d4334
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 3 deletions.
2 changes: 1 addition & 1 deletion ui/src/components/NavLink.jsx
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,7 @@ export default React.forwardRef((props, ref) => {
</Link>
);
} else {
const href = absolutePath ? url.toString() : cleanDuplicateSlash(getBasename() + "/" + url.toString());
const href = absolutePath ? url.toString() : cleanDuplicateSlash(getBasename() + url.toString());
return (
<Link ref={ref} target="_blank" href={href}>
{rest.children}
Expand Down
2 changes: 1 addition & 1 deletion ui/src/plugins/AppLogo.jsx
Original file line number Diff line number Diff line change
Expand Up @@ -13,6 +13,6 @@ const useStyles = makeStyles((theme) => ({

export default function AppLogo() {
const classes = useStyles();
const logoPath = getBasename() + '/logo.svg';
const logoPath = getBasename() + 'logo.svg';
return <img src={cleanDuplicateSlash(logoPath)} alt="Conductor" className={classes.logo} />;
}
2 changes: 1 addition & 1 deletion ui/src/plugins/fetch.js
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@ export function fetchWithContext(
const newParams = { ...fetchParams };

const basename = getBasename();
const newPath = basename + `/api/${path}`;
const newPath = basename + `api/${path}`;
const cleanPath = cleanDuplicateSlash(newPath); // Cleanup duplicated slashes

return fetch(cleanPath, newParams)
Expand Down

0 comments on commit 41d4334

Please sign in to comment.