dubbo icon indicating copy to clipboard operation
dubbo copied to clipboard

[Feature][3.3] Enhance JsonCompatibilityUtil to support triple protocol

Open AlbumenJ opened this issue 1 year ago • 6 comments

Pre-check

  • [X] I am sure that all the content I provide is in English.

Search before asking

  • [X] I had searched in the issues and found no similar feature requirement.

Apache Dubbo Component

Java SDK (apache/dubbo)

Descriptions

Enhance JsonCompatibilityUtil to support triple protocol none-annotation mode (application/json)

  • Check all the type that Dubbo can support
  • Generate the demo json struct
  • Log warn to notify users that this object might to compatiable with triple none-annotation mode

Related issues

No response

Are you willing to submit a pull request to fix on your own?

  • [ ] Yes I am willing to submit a pull request on my own!

Code of Conduct

AlbumenJ avatar Mar 20 '24 03:03 AlbumenJ

I can try solve it, please assign to me

hengyoush avatar Mar 22 '24 08:03 hengyoush

@hengyoush Go ahead

CrazyHZM avatar Mar 25 '24 03:03 CrazyHZM

Before I start, I have some questions about this issue:

  • Check all the types that Dubbo can support: I guess the current JsonCompatibilityUtil already has the functionality to check the compatibility of classes. Is it the same usage as JsonCompatibilityUtil in RestProtocol, and should we also use JsonCompatibilityUtil to do the same thing in TripleProtocol?
// this is the JsonCompatibilityUtil in RestProtocol
private void checkJsonCompatibility(Class<?> clazz, String jsonCheckLevel) throws RpcException {

        if (jsonCheckLevel == null || JSON_CHECK_LEVEL_WARN.equals(jsonCheckLevel)) {
        //....
        List<String> unsupportedMethods = JsonCompatibilityUtil.getUnsupportedMethods(clazz);
        } else if (JSON_CHECK_LEVEL_STRICT.equals(jsonCheckLevel)) {
        // ...
        }
    }
  • Generate the demo json struct: Does JsonCompatibilityUtil need to generate corresponding sample JSON structures based on interface parameters during the validation process?

hengyoush avatar Apr 01 '24 12:04 hengyoush

Before I start, I have some questions about this issue:

  • Check all the types that Dubbo can support: I guess the current JsonCompatibilityUtil already has the functionality to check the compatibility of classes. Is it the same usage as JsonCompatibilityUtil in RestProtocol, and should we also use JsonCompatibilityUtil to do the same thing in TripleProtocol?
// this is the JsonCompatibilityUtil in RestProtocol
private void checkJsonCompatibility(Class<?> clazz, String jsonCheckLevel) throws RpcException {

        if (jsonCheckLevel == null || JSON_CHECK_LEVEL_WARN.equals(jsonCheckLevel)) {
        //....
        List<String> unsupportedMethods = JsonCompatibilityUtil.getUnsupportedMethods(clazz);
        } else if (JSON_CHECK_LEVEL_STRICT.equals(jsonCheckLevel)) {
        // ...
        }
    }
  • Generate the demo json struct: Does JsonCompatibilityUtil need to generate corresponding sample JSON structures based on interface parameters during the validation process?
  • In TripleProtocol
  • No. This is a explict feature. @oxsean Do you have any ideas?

AlbumenJ avatar Apr 02 '24 06:04 AlbumenJ

Before I start, I have some questions about this issue:

  • Check all the types that Dubbo can support: I guess the current JsonCompatibilityUtil already has the functionality to check the compatibility of classes. Is it the same usage as JsonCompatibilityUtil in RestProtocol, and should we also use JsonCompatibilityUtil to do the same thing in TripleProtocol?
// this is the JsonCompatibilityUtil in RestProtocol
private void checkJsonCompatibility(Class<?> clazz, String jsonCheckLevel) throws RpcException {

        if (jsonCheckLevel == null || JSON_CHECK_LEVEL_WARN.equals(jsonCheckLevel)) {
        //....
        List<String> unsupportedMethods = JsonCompatibilityUtil.getUnsupportedMethods(clazz);
        } else if (JSON_CHECK_LEVEL_STRICT.equals(jsonCheckLevel)) {
        // ...
        }
    }
  • Generate the demo json struct: Does JsonCompatibilityUtil need to generate corresponding sample JSON structures based on interface parameters during the validation process?
  • In TripleProtocol
  • No. This is a explict feature. @oxsean Do you have any ideas?

I have completed the 1. and 3, but I am still confused about the second(Generate the demo json struct). Assuming we have a class like:

 public class DemoClass {
      private String str;
      private Set<String> set;
  }

and get a String through String demoStruct = JsonCompatibilityUtil.generateDemoJsonStruct(Class clazz):

{
	"str": "foobar",
	"set": ["v1", "v2"]
}

Is my understanding correct?

hengyoush avatar Apr 08 '24 12:04 hengyoush

@AlbumenJ Sorry for later to noticed this Should the focus be on leveraging JsonCompatibilityUtil in TripleProtocol to check compatibility? This should already be available. Generating the demo json structure may not be needed currently, but it could be useful for future support of OpenAPI.

oxsean avatar May 10 '24 15:05 oxsean