Skip to content

Minor changes to make code deployable on latest Cloudflare workers#24

Open
danieroux wants to merge 1 commit intoZibri:masterfrom
danieroux:master
Open

Minor changes to make code deployable on latest Cloudflare workers#24
danieroux wants to merge 1 commit intoZibri:masterfrom
danieroux:master

Conversation

@danieroux
Copy link

Fixes:

  • [ERROR] Cannot assign to "responseHeaders" because it is a constant
  • [WARNING] wrangler publish is deprecated and will be removed in the next major version.

Fixes:

- [ERROR] Cannot assign to "responseHeaders" because it is a constant
- [WARNING] `wrangler publish` is deprecated and will be removed in the next major version.
@niceEli
Copy link

niceEli commented May 25, 2024

looks good, you should request review

Copy link

@PianothShaveck PianothShaveck left a comment

Choose a reason for hiding this comment

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

I manually did these changes too

@AdamsGH
Copy link

AdamsGH commented May 28, 2024

Works good for me, thanks!

@EsmailELBoBDev2
Copy link

you are a life saver thank you so much

@Zibri can you merge this ?

Jediweirdo added a commit to Jediweirdo/cloudflare-cors-anywhere that referenced this pull request Dec 22, 2025
- Pass in a "x-cancel-redirect" custom header and set it to a true value to stop and log redirect behavior instead of automatically following
- Fixes compiling const error echoed in #39, Zibri#29, and Zibri#24
- Fixes bug that breaks URL-encoded URLs passed into the function by accidentally decoding them as addressed in Zibri#30

Later commits may expand on redirect tracking to add infinite loop protections
Jediweirdo added a commit to Jediweirdo/cloudflare-cors-anywhere that referenced this pull request Dec 22, 2025
- Fix depreciated deployment command reference (Zibri#24)
- Fixes "Cloudflaer" Typo in deployment section (Zibri#31)
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.

5 participants