Skip to content
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.

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

In Windows it generates an empty file #46

Closed
SyedAsimAliSE opened this issue Nov 7, 2022 · 5 comments · Fixed by #48
Closed

In Windows it generates an empty file #46

SyedAsimAliSE opened this issue Nov 7, 2022 · 5 comments · Fixed by #48
Assignees
Labels
question Further information is requested

Comments

@SyedAsimAliSE
Copy link

Did a fresh install of Deno , ran the server locally - and getting only this empty file am I missing something here ?

image

@nohehf
Copy link
Member

nohehf commented Nov 14, 2022

Could you please retry using powershell rather than cmd?
Are you sure you don't have any file at ~/Desktop/out/*.postman_collection.json?

@nohehf nohehf added the question Further information is requested label Nov 14, 2022
@nohehf nohehf self-assigned this Nov 14, 2022
@SyedAsimAliSE
Copy link
Author

Yes Just tried with Power Shell and no luck, it just generates this empty localhost file.

image

@nohehf
Copy link
Member

nohehf commented Nov 22, 2022

Ok, I believe the issue might come from the parsing of the : in the file name. Could you try to expose your api on the default HTTPS port and retry (so you'd have an address without :)? Could you also run the example on the rick and morty API (see README), to confirm it's not a windows related issue? If confirmed it will be an easy parse fix. However, if not I cannot currently try it on a windows machine. Thanks :)

@SyedAsimAliSE
Copy link
Author

Yes indeed it's the : parsing thing, with out port number and : it is working perfect, and thanks man it saved me bunch of time writing long ass queries :D

image

@nohehf
Copy link
Member

nohehf commented Nov 25, 2022

Glad it worked, will fix it today -> I'll add better parsing. My bad, this should have been working, but glad graphman helped you!

@nohehf nohehf mentioned this issue Nov 27, 2022
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
question Further information is requested
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants