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.

Sign up for GitHub

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

Jump to bottom

fix the failure to parse req.swagger.query #379

Open

lanchongyizu wants to merge 1 commit into RackHD:master from lanchongyizu:bugfix/swagger-query

Member

lanchongyizu commented May 14, 2018 •

edited

Loading

The result of req.swagger.params as below is different than the original in https://github.com/RackHD/on-http/blob/master/lib/services/swagger-api-service.js.

  [mac:
   { path: [ 'paths', '/profiles', 'get', 'parameters', '2' ],
     schema:
      { description: 'When macs parameter is not passed, passed with IP adds MAC address to lookup',
        in: 'query',
        name: 'mac',
        required: false,
        type: 'string' },
     originalValue: undefined,
     value: undefined }]

@RackHD/corecommitters @iceiilin @pengz1

lanchongyizu added the run-test label

iceiilin approved these changes

View reviewed changes


          fix the failure to parse req.swagger.query

e6d5932

lanchongyizu force-pushed the bugfix/swagger-query branch from 39712e4 to e6d5932 Compare

May 15, 2018 06:18

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels