Skip to content

Commit

Permalink
refactor: type event.node.req.originalUrl
Browse files Browse the repository at this point in the history
  • Loading branch information
pi0 committed Jul 31, 2023
1 parent e155950 commit 6c87d87
Show file tree
Hide file tree
Showing 3 changed files with 6 additions and 14 deletions.
6 changes: 2 additions & 4 deletions src/app.ts
Original file line number Diff line number Diff line change
Expand Up @@ -103,10 +103,8 @@ export function createAppEventHandler(stack: Stack, options: AppOptions) {
const spacing = options.debug ? 2 : undefined;
return eventHandler(async (event) => {
// Keep original incoming url accessable
(event.node.req as { originalUrl?: string }).originalUrl =
(event.node.req as { originalUrl?: string }).originalUrl ||
event.node.req.url ||
"/";
event.node.req.originalUrl =
event.node.req.originalUrl || event.node.req.url || "/";

// Keep a copy of incoming url
const _reqPath = event._path || event.node.req.url || "/";
Expand Down
8 changes: 2 additions & 6 deletions src/event/event.ts
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,7 @@ const PayloadMethods: Set<HTTPMethod> = new Set([
]);

export interface NodeEventContext {
req: NodeIncomingMessage;
req: NodeIncomingMessage & { originalUrl?: string };
res: NodeServerResponse;
}

Expand Down Expand Up @@ -49,11 +49,7 @@ export class H3Event<
}

get _originalPath() {
return (
(this.node.req as { originalUrl?: string }).originalUrl ||
this.node.req.url ||
"/"
);
return this.node.req.originalUrl || this.node.req.url || "/";
}

get _hasBody() {
Expand Down
6 changes: 2 additions & 4 deletions src/utils/route.ts
Original file line number Diff line number Diff line change
Expand Up @@ -11,10 +11,8 @@ export function useBase(base: string, handler: EventHandler): EventHandler {

return eventHandler(async (event) => {
// Keep original incoming url accessable
(event.node.req as { originalUrl?: string }).originalUrl =
(event.node.req as { originalUrl?: string }).originalUrl ||
event.node.req.url ||
"/";
event.node.req.originalUrl =
event.node.req.originalUrl || event.node.req.url || "/";

const _path = event._path || event.node.req.url || "/";

Expand Down

0 comments on commit 6c87d87

Please sign in to comment.