Allow more than 1 swagger per application

[swagger-api]

Allow more than 1 swagger per application

[paulhill]

I had the same issue.
Initially I used wrote this: https://www.npmjs.org/package/named-require
Then I rewrote the core of the code as an object. (It's really not a lot of
code).
I haven't had time to do the same to the open project unfortunately.

On Sun, Feb 23, 2014 at 9:16 AM, azuanagames [email protected]:

The application I am developing requires two different api definitions
that live on different ports. I currently have no way, that I can figure
out, to do this with swagger. Effectively I need to create two express
applications, but swagger has it's variables defined globally and does not
actual have a "Swagger" object. I believe the solution would be to create a
Swagger object that has the globals as members. I am still learning
Javascript myself so it's beyond my expertise. Further it seems like a huge
change...

Reply to this email directly or view it on GitHubhttps://github.com//issues/97
.

[azuanagames]

Thank you, I'll definitely give that a try! My biggest concern is wrapping it in an object and then not being able to pull in changes to the swagger-node-express module.

[azuanagames]

Your named-require did the trick for me. Thank you very much!

[jsdevel]

#111 seems like it would solve this for others as well.

@azuanagames @paulhill can you guys give that PR some +1s?

[fehguy]

I'll test and try to get this merged this weekend. I agree it's a great pr

[jsdevel]

Thanks @fehguy! That would be great. Just to give you some background, @enterline and myself both work at GoDaddy and we're building one of the first public facing APIs herein using swagger-node-express. If you can get this merged in soon it would really give us the warm and fuzzies and would also give us incentive to spread the word internally.

Up to this point we've been using @enterline's fork due this missing functionality. We'd rather reference npm :(

[fehguy]

hi @jsdevel this is merged in #111 with a couple minor fixes.

[jsdevel]

Thanks @fehguy!