Skip to content

Commit

Permalink
fix: allow http for production prax too (#208)
Browse files Browse the repository at this point in the history
So this issue was stuck for some time. I retested it again and the minifront seem to be working correctly with the change. 

We have a user who got stuck because of this: penumbra-zone/nextjs-penumbra-client-example#10
  • Loading branch information
VanishMax authored Dec 2, 2024
1 parent 23f6d3f commit 081f8f9
Showing 1 changed file with 1 addition and 6 deletions.
7 changes: 1 addition & 6 deletions apps/extension/src/senders/validate.ts
Original file line number Diff line number Diff line change
Expand Up @@ -35,12 +35,7 @@ export const assertValidSender = (sender?: chrome.runtime.MessageSender) => {
throw new Error('Sender origin is invalid');
}

if (
!(
parsedOrigin.protocol in ValidProtocol ||
(globalThis.__DEV__ && isHttpLocalhost(parsedOrigin))
)
) {
if (!(parsedOrigin.protocol in ValidProtocol || isHttpLocalhost(parsedOrigin))) {
throw new Error(`Sender protocol is not ${Object.values(ValidProtocol).join(',')}`);
}

Expand Down

0 comments on commit 081f8f9

Please sign in to comment.