Skip to content
This repository was archived by the owner on Nov 4, 2021. It is now read-only.

Remove Fastify and web server - #282

Closed
Twixes wants to merge 1 commit into
masterfrom
no-web
Closed

Remove Fastify and web server#282
Twixes wants to merge 1 commit into
masterfrom
no-web

Conversation

@Twixes

@Twixes Twixes commented Mar 24, 2021

Copy link
Copy Markdown
Member

Changes

We've realized it's not feasible to run a plugin web server due to the limitations of Heroku (which is an important deployment platform for us): https://github.com/PostHog/plugin-server/issues/165#issuecomment-805759098
Therefore this PR removes the Fastify web server that has been dead code for a while.
Closes #25.

Checklist

  • Updated Settings section in README.md, if settings are affected
  • Jest tests

@Twixes
Twixes requested a review from mariusandra March 24, 2021 13:39

@macobo macobo left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Yeah, tried running it recently and was quite confused why / returned an error. Thanks!

@mariusandra

Copy link
Copy Markdown
Collaborator

We haven't moved far with developments around this, but I'd keep it in for longer. At least until we get to do a proof of concept of running the plugin server inside the web dyno in heroku and seeing if we can get some communication between them.

@Twixes

Twixes commented May 27, 2021

Copy link
Copy Markdown
Member Author

Superseded by #439.

@Twixes Twixes closed this May 27, 2021
@Twixes
Twixes deleted the no-web branch May 27, 2021 11:10
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants