Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Feature Request] Replace proxy server with Next.js #646

Open
kmcginnes opened this issue Oct 11, 2024 · 0 comments
Open

[Feature Request] Replace proxy server with Next.js #646

kmcginnes opened this issue Oct 11, 2024 · 0 comments
Labels
enhancement New feature or request infrastructure performance Issues relating to performance usability Issues relating to the ease of use of the UI or features

Comments

@kmcginnes
Copy link
Collaborator

kmcginnes commented Oct 11, 2024

Description

Replace the proxy server with Next.js and move the data processing logic to the server.

Additional Context

Using Next.js has both advantages and disadvantages.

Advantages

  • User focused
    • No proxy server configuration in connection
    • Faster startup performance
    • Less data processing on user's machine
    • Less data sent to client resulting in less bandwidth used
    • Elimination of confusing edge cases related to proxy server configuration in connection
  • Infrastructure focused
    • Monorepo can be collapsed in to a single package
    • Better type safety and code sharing across client and server
    • Guaranteed server availability
    • Reduced deployment configuration
    • Provides a foundation for adding a shared centralized database for Graph Explorer data

Disadvantages

  • HTTP paths will change, requiring some users to change their network configurations
  • More difficult to embed in to a desktop app

Alternatives considered

Remix

Great performance and the focus on web standards is fantastic. But Next offers some great routing features that will be useful for Graph Explorer. Next also has support for React Server Components and Server Actions.

tRPC

tRPC provides some nice benefits around type safety, but doesn't allow the removal of the proxy server.

Related Issues


Important

If you are interested in working on this issue or have submitted
a pull request, please leave a comment.

Tip

Please use a 👍 reaction to provide a +1/vote.

This helps the community and maintainers prioritize this request.

@kmcginnes kmcginnes added enhancement New feature or request performance Issues relating to performance usability Issues relating to the ease of use of the UI or features infrastructure labels Oct 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request infrastructure performance Issues relating to performance usability Issues relating to the ease of use of the UI or features
Projects
None yet
Development

No branches or pull requests

1 participant