kuzznya / exposer Goto Github PK

Check if all method params are mapped & throw exception if not
or use default principle (params['<name>'] ?: pathVars['<name'])

Handle Set & other types of collections

Now the attempt causes IllegalArgumentException

EndpointPropertyBuilder sets empty map for endpoint params leading to error with EndpointHandler when it tries to get value for key

Set null instead of empty map

Syntax $(...) is redundant
Better use : params['val'] without $(...)

Originally posted by @kuzznya in #5 (comment)

• Enable args: params['somevalue'], pathVars['test'] syntax to simplify params definition (maybe too difficult)
• Add definition of what to do with null values - throw exception or call method with null

Add logging for each request & response, especially errors (if required, because Spring Boot also logs errors)

Recursive bean resolving for endpoints isn't working

Test cast from various params to method params, e.g. request param (always String) to double

EndpointHandler cannot load correct names of method it calls

Solution: use ParameterNameDiscoverer

Consider making it a bean to avoid creating it for each EndpointHandler

Add path parameter to endpoint configuration to enable defining some part of the path directly for the endpoint & simplify some config, e.g.:

exposer:
  endpoints:
    - path: /test/v1
       ...
    - path: /test/v2

For more complex request handling actions, such as getting data from body, new syntax is required

Some ideas:

params:
  arg0: textArg
  arg1: {{ params.someVal }}
  user: {{ body.username }}
  data: {{ body }}

• New syntax
• Split parsing & evaluation
• Request body handling
• Cover with tests

Use .getMethods() instead of .useDeclaredMethods() to allow use superclass methods

EndpointHandler cannot return non-String values because no JSON converter set for core module