oVirt Jenkins CI Server has posted comments on this change.
Change subject: services: add proxy servlet
..
Patch Set 10:
Build Successful
http://jenkins.ovirt.org/job/ovirt-engine_master_dao-unit-tests_merged/8664/ :
SUCCESS
Oved Ourfali has submitted this change and it was merged.
Change subject: services: add proxy servlet
..
services: add proxy servlet
a proxy servlet that proxy into url appending query string.
example of usage:
google1
o
automat...@ovirt.org has posted comments on this change.
Change subject: services: add proxy servlet
..
Patch Set 10:
* Update tracker::IGNORE, no Bug-Url found
* Set MODIFIED::IGNORE, no Bug-Url found.
--
To view, visit http
oVirt Jenkins CI Server has posted comments on this change.
Change subject: services: add proxy servlet
..
Patch Set 9:
Build Failed
http://jenkins.ovirt.org/job/ovirt_engine_master_compile_checkstyle_gerrit/32245/
: There w
automat...@ovirt.org has posted comments on this change.
Change subject: services: add proxy servlet
..
Patch Set 9:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matches '
oVirt Jenkins CI Server has posted comments on this change.
Change subject: services: add proxy servlet
..
Patch Set 8:
Build Successful
http://jenkins.ovirt.org/job/ovirt_engine_master_compile_checkstyle_gerrit/31777/
: SUC
automat...@ovirt.org has posted comments on this change.
Change subject: services: add proxy servlet
..
Patch Set 8:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matches '
oVirt Jenkins CI Server has posted comments on this change.
Change subject: services: add proxy servlet
..
Patch Set 7: -Code-Review -Verified
Build Successful
http://jenkins.ovirt.org/job/ovirt_engine_master_compile_checksty
Alon Bar-Lev has posted comments on this change.
Change subject: services: add proxy servlet
..
Patch Set 7: -Code-Review Verified+1
--
To view, visit http://gerrit.ovirt.org/30740
To unsubscribe, visit http://gerrit.ovirt.org
oVirt Jenkins CI Server has posted comments on this change.
Change subject: services: add proxy servlet
..
Patch Set 7: Code-Review-1 Verified-1
Build Unstable
http://jenkins.ovirt.org/job/ovirt_engine_master_compile_checksty
automat...@ovirt.org has posted comments on this change.
Change subject: services: add proxy servlet
..
Patch Set 7:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matches '
Alon Bar-Lev has posted comments on this change.
Change subject: services: add proxy servlet
..
Patch Set 6: Code-Review-1
please do not submit until the complete series is ready.
--
To view, visit http://gerrit.ovirt.org/307
Oved Ourfali has posted comments on this change.
Change subject: services: add proxy servlet
..
Patch Set 6: Code-Review+2
--
To view, visit http://gerrit.ovirt.org/30740
To unsubscribe, visit http://gerrit.ovirt.org/settings
Ravi Nori has posted comments on this change.
Change subject: services: add proxy servlet
..
Patch Set 6: Verified+1 Code-Review+1
--
To view, visit http://gerrit.ovirt.org/30740
To unsubscribe, visit http://gerrit.ovirt.org/s
Alexander Wels has posted comments on this change.
Change subject: services: add proxy servlet
..
Patch Set 6:
Right that is what I mean, we are not proxying the body of the request, which
makes POST/PUT a little hard to suppo
Alon Bar-Lev has posted comments on this change.
Change subject: services: add proxy servlet
..
Patch Set 6:
> And I take it thats a NO on proxying anything but GETs
well, more correctly, the body of request is not proxied, I
Alexander Wels has posted comments on this change.
Change subject: services: add proxy servlet
..
Patch Set 6: Code-Review+1
Fair enough. I checked and There are only 2 subclasses of UrlConnection,
HttpUrlConnection and JarUrl
Alon Bar-Lev has posted comments on this change.
Change subject: services: add proxy servlet
..
Patch Set 6:
> Looks much better IMO, now I have 2 questions:
well, I wanted to leave this as abstract as I could, without trying
Alexander Wels has posted comments on this change.
Change subject: services: add proxy servlet
..
Patch Set 6:
Looks much better IMO, now I have 2 questions:
1. Maybe we should simply use HttpUrlConnection instead of URLConnec
oVirt Jenkins CI Server has posted comments on this change.
Change subject: services: add proxy servlet
..
Patch Set 6:
Build Failed
http://jenkins.ovirt.org/job/ovirt_engine_master_compile_checkstyle_gerrit/30244/
: SUCCESS
automat...@ovirt.org has posted comments on this change.
Change subject: services: add proxy servlet
..
Patch Set 6:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matches '
Alexander Wels has posted comments on this change.
Change subject: services: add proxy servlet
..
Patch Set 5:
(1 comment)
http://gerrit.ovirt.org/#/c/30740/5/backend/manager/modules/uutils/src/main/java/org/ovirt/engine/core/
Alexander Wels has posted comments on this change.
Change subject: services: add proxy servlet
..
Patch Set 4:
(1 comment)
http://gerrit.ovirt.org/#/c/30740/4/backend/manager/modules/uutils/src/main/java/org/ovirt/engine/core/
oVirt Jenkins CI Server has posted comments on this change.
Change subject: services: add proxy servlet
..
Patch Set 5:
Build Successful
http://jenkins.ovirt.org/job/ovirt_engine_master_compile_checkstyle_gerrit/30240/
: SUC
automat...@ovirt.org has posted comments on this change.
Change subject: services: add proxy servlet
..
Patch Set 5:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matches '
Alon Bar-Lev has posted comments on this change.
Change subject: services: add proxy servlet
..
Patch Set 4:
> Any unit tests for this?
no... have no time for this... maybe rnori can prepare if this is important.
--
To view,
Alon Bar-Lev has posted comments on this change.
Change subject: services: add proxy servlet
..
Patch Set 4:
(5 comments)
http://gerrit.ovirt.org/#/c/30740/4/backend/manager/modules/services/src/main/java/org/ovirt/engine/core
Alexander Wels has posted comments on this change.
Change subject: services: add proxy servlet
..
Patch Set 4:
(5 comments)
Any unit tests for this?
http://gerrit.ovirt.org/#/c/30740/4/backend/manager/modules/services/src/mai
oVirt Jenkins CI Server has posted comments on this change.
Change subject: services: add proxy servlet
..
Patch Set 4:
Build Successful
http://jenkins.ovirt.org/job/ovirt_engine_master_compile_checkstyle_gerrit/30234/
: SUC
automat...@ovirt.org has posted comments on this change.
Change subject: services: add proxy servlet
..
Patch Set 4:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matches '
Alon Bar-Lev has posted comments on this change.
Change subject: services: add proxy servlet
..
Patch Set 4:
added serialization magics to make code scanners happy
--
To view, visit http://gerrit.ovirt.org/30740
To unsubscrib
oVirt Jenkins CI Server has posted comments on this change.
Change subject: services: add proxy servlet
..
Patch Set 3:
Build Successful
http://jenkins.ovirt.org/job/ovirt_engine_master_compile_checkstyle_gerrit/30228/
: SUC
Alon Bar-Lev has posted comments on this change.
Change subject: services: add proxy servlet
..
Patch Set 3:
return not found if no url is configured, to make url optional as well.
--
To view, visit http://gerrit.ovirt.org/30
automat...@ovirt.org has posted comments on this change.
Change subject: services: add proxy servlet
..
Patch Set 3:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matches '
Oved Ourfali has posted comments on this change.
Change subject: services: add proxy servlet
..
Patch Set 2: Code-Review+1
--
To view, visit http://gerrit.ovirt.org/30740
To unsubscribe, visit http://gerrit.ovirt.org/settings
Oved Ourfali has posted comments on this change.
Change subject: services: add proxy servlet
..
Patch Set 1: Code-Review+1
--
To view, visit http://gerrit.ovirt.org/30740
To unsubscribe, visit http://gerrit.ovirt.org/settings
automat...@ovirt.org has posted comments on this change.
Change subject: services: add proxy servlet
..
Patch Set 2:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matches '
Alon Bar-Lev has posted comments on this change.
Change subject: services: add proxy servlet
..
Patch Set 2:
expose readTimeout as well
--
To view, visit http://gerrit.ovirt.org/30740
To unsubscribe, visit http://gerrit.ovirt
oVirt Jenkins CI Server has posted comments on this change.
Change subject: services: add proxy servlet
..
Patch Set 2:
Build Failed
http://jenkins.ovirt.org/job/ovirt_engine_master_compile_checkstyle_gerrit/30083/
: SUCCESS
oVirt Jenkins CI Server has posted comments on this change.
Change subject: services: add proxy servlet
..
Patch Set 1:
Build Failed
http://jenkins.ovirt.org/job/ovirt_engine_master_compile_checkstyle_gerrit/30078/
: SUCCESS
Alon Bar-Lev has uploaded a new change for review.
Change subject: services: add proxy servlet
..
services: add proxy servlet
a proxy servlet that proxy into url appending query string.
example of usage:
google1
org.ovirt
automat...@ovirt.org has posted comments on this change.
Change subject: services: add proxy servlet
..
Patch Set 1:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matches '
42 matches
Mail list logo
