[Feature] Add Unwrap to custom response writer. about kratos HOT 3 CLOSED

It's great to see your interest in adding the Unwrap method to the custom ResponseWriter. Your proposed solution and the requirements description look good. We encourage you to go ahead and create a pull request with your changes. Your contribution will be valuable to the project. Thank you for your willingness to improve the codebase!

from kratos.

Sure, I can guide you through the process of creating a pull request to add an "Unwrap" method to the custom ResponseWriter in the Kratos repository. Here are the steps:

1. Fork the Repository: First, you need to fork the Kratos repository to your own GitHub account.

2. Clone the Forked Repository: After forking, clone the repository to your local machine using the following commands:

   git clone https://github.com/your-username/kratos.git
   cd kratos

3. Create a New Branch: It's a good practice to create a new branch for your changes. You can do this using the following command:

   git checkout -b add-unwrap-method

4. Make Your Changes: Now, you can add the "Unwrap" method to the custom ResponseWriter in the transport/http package. Your proposed code looks correct. Just make sure to add it in the correct file and location.

5. Commit Your Changes: After making your changes, you need to commit them. You can do this using the following commands:

   git add .
   git commit -m "Add Unwrap method to custom ResponseWriter"

6. Push Your Changes: Next, you need to push your changes to your forked repository. You can do this using the following command:

   git push origin add-unwrap-method

7. Create a Pull Request: Finally, you can create a pull request. Go to your forked repository on GitHub, select your branch, and click on "New pull request". Then, fill in the necessary details and submit it.

Please note that after submitting your pull request, it will be reviewed by the maintainers of the Kratos repository. They might ask you to make some changes or improvements to your code. So, be prepared for that.

I hope this helps! If you have any other questions, feel free to ask.

Sources

• go.mod
• transport/http/transport.go

from kratos.

It's great to see your interest in adding the Unwrap method to the custom ResponseWriter. Your proposed solution and the requirements description look good. We encourage you to go ahead and create a pull request with your changes. Your contribution will be valuable to the project. Thank you for your willingness to improve the codebase!

About Dosu

This response is meant to be useful and save you time. It is not meant to be a precise solution, but rather a starting point for your own research.

Help me be more useful! Please leave a 👍 if this is helpful and 👎 if it is irrelevant.

If you want to continue the conversation, start your reply with @dosu-bot.

@kvii

from kratos.