Skip to content

feat(cache): add healthcheck endpoint - #53

Merged
Svenum merged 2 commits into
mainfrom
fix-cache-index.ts
Oct 14, 2026
Merged

Svenum merged 2 commits into
mainfrom
fix-cache-index.ts

Conversation

@Svenum

@Svenum Svenum commented Oct 14, 2026

Copy link
Copy Markdown
Member

No description provided.

@Svenum
Svenum requested a review from SirBerg October 14, 2026 10:34
@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Oct 14, 2026 •

Copy link
Copy Markdown

Deploying iglu with  Cloudflare Pages  Cloudflare Pages

Latest commit: f37cd85
Status: ✅  Deploy successful!
Preview URL: https://4e78a2b1.iglu.pages.dev
Branch Preview URL: https://fix-cache-index-ts.iglu.pages.dev

View logs

@SirBerg SirBerg left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This sets the response to 200 for every catchall endpoint. Create a Healthcheck endpoint if you want to check if the cache is healthy or not

@Svenum

Svenum commented Oct 14, 2026

Copy link
Copy Markdown
Member Author

I noticed that as well XD

@SirBerg

SirBerg commented Oct 14, 2026

Copy link
Copy Markdown
Member

Yeah maybe create an endpoint at api/v1/iglu/healthcheck and try that

@Svenum
Svenum force-pushed the fix-cache-index.ts branch from 6eda2e3 to 67297a1 Compare October 14, 2026 10:46
@Svenum

Svenum commented Oct 14, 2026

Copy link
Copy Markdown
Member Author

I want to have / return a 200. This is the most easy way to check.

Comment thread cache/index.ts Outdated
@Svenum
Svenum force-pushed the fix-cache-index.ts branch from 67297a1 to a37bdb2 Compare October 14, 2026 11:57
@Svenum
Svenum requested a review from SirBerg October 14, 2026 11:58
@Svenum
Svenum force-pushed the fix-cache-index.ts branch from a37bdb2 to e3bc93c Compare October 14, 2026 12:01

@SirBerg SirBerg left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Will approve this for now but please take a look at the things I comment and maybe change them if you feel like it

Comment thread cache/routes/api/v1/iglu/healthcheck.ts Outdated
Comment thread cache/routes/api/v1/iglu/healthcheck.ts
@SirBerg

SirBerg commented Oct 14, 2026

Copy link
Copy Markdown
Member

Love the docusaurus versioning btw

@Svenum
Svenum force-pushed the fix-cache-index.ts branch from e3bc93c to 0a0ebd2 Compare October 14, 2026 13:55
@Svenum
Svenum force-pushed the fix-cache-index.ts branch from 0a0ebd2 to f37cd85 Compare October 14, 2026 14:03
@Svenum Svenum changed the title fix: set response of / to 200 for healthcheck reasons feat(cache): add healthcheck endpoint Oct 14, 2026
@Svenum
Svenum merged commit 11bbc67 into main Oct 14, 2026
8 checks passed
@Svenum
Svenum deleted the fix-cache-index.ts branch October 14, 2026 14:13
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.

2 participants