Change PetascopeInterface parent

[Vlad Merticariu]

Refactor PetascopeInterface to use the functionality for CORS request available in CORSHttpServlet class, once available.

[Dimitar Misev]

Vlad please set milestone and component when you open tickets!

[Dimitar Misev]

can be closed?

[Bang Pham Huu]

this is not fixed yet with change in PetascopeInterface to "extends CORSHttpServlet", I think I will take it.

[Bang Pham Huu]

patch was accepted, close ticket.

[Alex Dumitru]

The patch broke CORS functionality.
You need to call super.doGet() in PetascopeInterface.doGet() before doing anything else in the function

[Bang Pham Huu]

Replying to mdumitru:

The patch broke CORS functionality.
You need to call super.doGet() in PetascopeInterface.doGet() before doing anything else in the function

cool, I will make a fix for it from your hint now.

[Dimitar Misev]

There was a test in the patch, what is it testing if not the CORS functionality??

[Bang Pham Huu]

Replying to dmisev:

There was a test in the patch, what is it testing if not the CORS functionality??

it simulate a request from a random host to Petascope and check in the HTTP header, it has the "Allow origin: *".

[Dimitar Misev]

But obviously that test doesn't work correctly if Alex says the CORS functionality is broken now.

[Dimitar Misev]

I mean the test still passes, so it's incorrect.

[Bang Pham Huu]

Replying to dmisev:

But obviously that test doesn't work correctly if Alex says the CORS functionality is broken now.

ok, I will give more time to make the test correctly.

[Bang Pham Huu]

the patch was accepted, thanks.