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.

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

[BrowsermobProxy] Remove as many unused code from the lib as possible #14

Closed
tkohegyi opened this issue Mar 9, 2015 · 2 comments
Closed

[BrowsermobProxy] Remove as many unused code from the lib as possible #14

tkohegyi opened this issue Mar 9, 2015 · 2 comments
Assignees
@tkohegyi
Labels
enhancement

Comments

@tkohegyi
Copy link
Contributor

tkohegyi commented Mar 9, 2015

Investigate updates done in BrowserMob Proxy project, either update it in this project with the fixes - or if it is possible, - merge enhancements done here back to the original code base.
@tkohegyi tkohegyi changed the title Upgrade to latest BowserMob Proxy - as possible [BrowsermobProxy] Upgrade to latest BowserMob Proxy - as possible Jun 9, 2015
tkohegyi added a commit that referenced this issue Jul 23, 2015
@tkohegyi
- fix build/release for browsermob-proxy part, to prepare new release, …
9d029ca 
…#38, #14

- increase proxy timeout from 1 minute to 4 minutes (when in case of timeout, -998 error code is generated)
@tkohegyi tkohegyi self-assigned this Jul 23, 2015
tkohegyi added a commit that referenced this issue Jul 23, 2015
@tkohegyi
upgrade wilma to use latest browsermob-proxy module for wilma - #38, #14
999e56c
tkohegyi added a commit that referenced this issue Sep 27, 2015
@tkohegyi
- address #63 + in addition fix pretty print attempt of image/svg+xml…
49831fb 
… -where it is not necessary

- remove unnecessary gradle executables
- alter browsermob-proxy README.md to make it clear that this differs from the original lib, see #14
tkohegyi added a commit that referenced this issue Sep 27, 2015
@tkohegyi
- remove big amount of browsermob-proxy code base that is not used by…
7bcb3e9 
… Wilma - see #59 (considered as addressed) and #14 (ongoing)
@tkohegyi
Copy link
Contributor Author

Actually the aim is not to upgrade browsermob-proxy to its latest codebase, rather remove all unnecessary code (just see previous commit.)

@tkohegyi tkohegyi changed the title [BrowsermobProxy] Upgrade to latest BowserMob Proxy - as possible [BrowsermobProxy] Remove as many unused code from the lib as possible Sep 29, 2015
tkohegyi added a commit that referenced this issue Oct 2, 2015
@tkohegyi
preparing Wilma 1.2 release with new proxy - proxy part upgrade is pr…
b5b0f76 
…epared

- reduced content, optimizations in code and more separation from Wilma, addressing #14
- prepare eliminating further unnecessary code, see #14
- hardcoded timeout now eliminated, see #59
tkohegyi added a commit that referenced this issue Oct 5, 2015
@tkohegyi
Upgrading Wilma to use browsermob-proxy-for-wilma 1.2 version. The in…
4bfb074 
…terface of the proxy was changed and the possibility of optimizing the proxy code further was opened. See #14 and #59, those become available in Wilma with this upgrade.
@tkohegyi
Copy link
Contributor Author

tkohegyi commented Oct 9, 2015

ok, closing, will be released in V1.2.83 of Wilma

@tkohegyi tkohegyi closed this as completed Oct 9, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@tkohegyi tkohegyi

Labels
enhancement
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@tkohegyi