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

feat: common dubbo-proxy plugin #10848

Closed
mayunSaicmotor opened this issue Jan 18, 2024 · 7 comments
Closed

feat: common dubbo-proxy plugin #10848

mayunSaicmotor opened this issue Jan 18, 2024 · 7 comments
Assignees
Labels

Comments

@mayunSaicmotor
Copy link

Description

Dear,
It seems it is not easy to use the dubbo proxy plugin in real project. It has too much restriction. It's not very common solution.
Do you have plan to develop a common dubbo proxy plugin in future?

@sheharyaar
Copy link
Contributor

@mayunSaicmotor , as we discussed in the previous issue #10648 , I would summarise the issues we faced :

  • mod_dubbo used by APISIX, does not any datatype other than Map<String, Object> for both as return type and arguments.
  • mod_dubbo uses hessian2 as the only deserialization protocol
  • data passed through headers work out of the box, but for data in the body, it requires further deserialization using hessian2 decoder.

These are the issues that need to be worked upon to make dubbo-proxy better.

@Vacant2333
Copy link
Contributor

Agree! how do u think about this feat @shreemaan-abhishek @monkeyDluffy6017

@sheharyaar
Copy link
Contributor

@shreemaan-abhishek , can you please assign me this

@sheharyaar
Copy link
Contributor

Hi @mayunSaicmotor , can you please checkout this : #10703 , usage example can be found under t/plugin/http-dubbo.t

@monkeyDluffy6017
Copy link
Contributor

LGTM

Copy link

This issue has been marked as stale due to 350 days of inactivity. It will be closed in 2 weeks if no further activity occurs. If this issue is still relevant, please simply write any comment. Even if closed, you can still revive the issue at any time or discuss it on the [email protected] list. Thank you for your contributions.

@github-actions github-actions bot added the stale label Jan 19, 2025
Copy link

github-actions bot commented Feb 3, 2025

This issue has been closed due to lack of activity. If you think that is incorrect, or the issue requires additional review, you can revive the issue at any time.

@github-actions github-actions bot closed this as not planned Won't fix, can't repro, duplicate, stale Feb 3, 2025
@github-project-automation github-project-automation bot moved this from 📋 Backlog to ✅ Done in Apache APISIX backlog Feb 3, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Archived in project
Development

No branches or pull requests

4 participants