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

WIP: Exporting the workspace / all workspaces as HAR file #523

Merged
merged 3 commits into from Nov 1, 2017

Conversation

@Atvaark
Copy link
Contributor

Atvaark commented Oct 12, 2017

This PR adds a the option to export the requests in a workspace (or all workspaces) as an HAR (HTTP Archive 1.2) file.

See: #416

Exporting as .har is implemented.
Exporting single har requests still works as it looks (the code generator).
I'm currently testing the output in different tools (e. g. Fiddler or HTTPolice) to check for any issues.

Closes #416

@Atvaark Atvaark changed the title WIP: Implemented exporting the workspace / all workspaces as HAR file WIP: Exporting the workspace / all workspaces as HAR file Oct 12, 2017
@gschier

This comment has been minimized.

Copy link
Contributor

gschier commented Oct 12, 2017

Thanks for this @Atvaark! I'll try and get to reviewing/merging this tomorrow.

Copy link
Contributor

gschier left a comment

Hey @Atvaark. I just took a look at this and it looks great. However, since the code has quite a bit of complexity, it would be nice to see some tests around exportHarResponse as well as at least one test for exportHar to make sure nothing breaks in the future.

The current tests you touched in this PR should be a good starting point, but let me know if you need/want any additional guidance. 😄

}

export async function getLatestForRequest (requestId: string) {
export async function getLatestForRequest (requestId: string): Response | null {

This comment has been minimized.

Copy link
@gschier

gschier Oct 13, 2017

Contributor

Minor, but this should return Promise<Response | null>.

@Atvaark

This comment has been minimized.

Copy link
Contributor Author

Atvaark commented Oct 16, 2017

Hey @gschier,
I added a couple of tests. Please take a look at them in case I missed something.

@gschier

This comment has been minimized.

Copy link
Contributor

gschier commented Oct 16, 2017

Thanks, I will try to take a look today.

@gschier

This comment has been minimized.

Copy link
Contributor

gschier commented Oct 27, 2017

Sorry for the delay @Atvaark. I was without a computer for the last two weeks because my keyboard broke. I have this on my list of to-dos for next week, though. 😄

@gschier
gschier approved these changes Nov 1, 2017
Copy link
Contributor

gschier left a comment

This looks good! Going to merge it in and make a few small tweaks to help it match the codebase conventions a bit closer. I'll be including it in this week's release.

Thanks for the PR!

@gschier gschier merged commit f6d00d3 into Kong:develop Nov 1, 2017
2 checks passed
2 checks passed
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
luizmariz pushed a commit to luizmariz/insomnia that referenced this pull request Jan 22, 2020
* Implemented exporting the workspace / all workspaces as HAR file

* Added HAR export unit tests
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.