Skip to content

Fix Proxy Redirect Logic#7

Merged
browser-vm merged 1 commit intomainfrom
fine/fix-proxy-redirect
Feb 7, 2025
Merged

Fix Proxy Redirect Logic#7
browser-vm merged 1 commit intomainfrom
fine/fix-proxy-redirect

Conversation

@fine-ai
Copy link

@fine-ai fine-ai bot commented Feb 7, 2025

Refactor proxy handling with Ultraviolet in server.js and update uv.config.js

- Updated `server.js` to properly handle proxy requests using Ultraviolet:
  1. Imported necessary Ultraviolet components.
  2. Initialized Ultraviolet middleware for proxy handling.
  3. Replaced the outdated '/service/*' route handler with proper Ultraviolet initialization and request handling.
  4. Updated URL encoding/decoding logic to use Ultraviolet's built-in methods.

- Updated `uv.config.js` to properly configure Ultraviolet:
  1. Exported the configuration object correctly instead of assuming a global object.
  2. Updated the prefix to match the service endpoint.
  3. Configured the bare server endpoint.
  4. Set up proper encoding/decoding methods for consistency.

These changes fix incorrect usage of Ultraviolet and ensure proper proxy functionality.

```
Refactor proxy handling with Ultraviolet in server.js and update uv.config.js

- Updated `server.js` to properly handle proxy requests using Ultraviolet:
  1. Imported necessary Ultraviolet components.
  2. Initialized Ultraviolet middleware for proxy handling.
  3. Replaced the outdated '/service/*' route handler with proper Ultraviolet initialization and request handling.
  4. Updated URL encoding/decoding logic to use Ultraviolet's built-in methods.

- Updated `uv.config.js` to properly configure Ultraviolet:
  1. Exported the configuration object correctly instead of assuming a global object.
  2. Updated the prefix to match the service endpoint.
  3. Configured the bare server endpoint.
  4. Set up proper encoding/decoding methods for consistency.

These changes fix incorrect usage of Ultraviolet and ensure proper proxy functionality.
```
@sonarqubecloud
Copy link

sonarqubecloud bot commented Feb 7, 2025

@browser-vm browser-vm marked this pull request as ready for review February 7, 2025 12:29
@browser-vm browser-vm merged commit 636f66a into main Feb 7, 2025
4 checks passed
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.

1 participant