thriftcli icon indicating copy to clipboard operation
thriftcli copied to clipboard

Allow snake case parameter naming for service endpoints

Open alexandrustaetu opened this issue 6 years ago • 0 comments

It looks like the service endpoint parser will have trouble extracting methods that have snake case parameters

This is the current regex matching when presenting snake case parameters

alex@computer:~$ echo '    Service endpoint(1:Service some_service) throws (1: Exception exception),' | grep -P '^[\r\t ]*(oneway)?\s*([^\n]*)\s+(\w+)\(([a-zA-Z0-9: ,.<>]*)\)' || echo 'no matches found'

no matches found

This is the updated regex rule

alex@computer:~$ echo '    Service endpoint(1:Service some_service) throws (1: Exception exception),' | grep -P '^[\r\t ]*(oneway)?\s*([^\n]*)\s+(\w+)\(([a-zA-Z0-9: ,.<>_]*)\)' || echo 'no matches found'

    Service endpoint(1:Service some_service) throws (1: Exception exception),

alexandrustaetu avatar Sep 25 '19 10:09 alexandrustaetu