Skip to content

Conversation

@4cecoder
Copy link

@4cecoder 4cecoder commented Dec 6, 2024

Fixes #26

Add HTTP Proxy support to ClientOptions.

  • client.go

    • Add Proxy field to ClientOptions struct to configure HTTP Proxy.
    • Update NewClient function to use Proxy field if provided.
    • Update clientOptions struct to include proxy field.
    • Update NewClient function to set proxy field in clientOptions.
  • client_test.go

    • Add test cases to verify HTTP Proxy functionality in TestFrom.
    • Add test cases to verify HTTP Proxy functionality in TestRpc.
    • Add test cases to verify HTTP Proxy functionality in TestStorage.
    • Add test cases to verify HTTP Proxy functionality in TestFunctions.
  • README.md

    • Add documentation on how to configure HTTP Proxy in ClientOptions.
    • Add example code snippet for configuring HTTP Proxy.

For more details, open the Copilot Workspace session.

Fixes supabase-community#26

Add HTTP Proxy support to ClientOptions.

* **client.go**
  - Add `Proxy` field to `ClientOptions` struct to configure HTTP Proxy.
  - Update `NewClient` function to use `Proxy` field if provided.
  - Update `clientOptions` struct to include `proxy` field.
  - Update `NewClient` function to set `proxy` field in `clientOptions`.

* **client_test.go**
  - Add test cases to verify HTTP Proxy functionality in `TestFrom`.
  - Add test cases to verify HTTP Proxy functionality in `TestRpc`.
  - Add test cases to verify HTTP Proxy functionality in `TestStorage`.
  - Add test cases to verify HTTP Proxy functionality in `TestFunctions`.

* **README.md**
  - Add documentation on how to configure HTTP Proxy in `ClientOptions`.
  - Add example code snippet for configuring HTTP Proxy.

---

For more details, open the [Copilot Workspace session](https://copilot-workspace.githubnext.com/supabase-community/supabase-go/issues/26?shareId=XXXX-XXXX-XXXX-XXXX).
Copy link

@0x113 0x113 left a comment

Choose a reason for hiding this comment

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

Looks good to me, but I guess we need to wait for the contributors to take a look.

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