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

Transform schemaLocation in WsdlDefinitionHandlerAdapter [SWS-791] #873

Closed
gregturn opened this issue Aug 8, 2012 · 24 comments
Closed

Transform schemaLocation in WsdlDefinitionHandlerAdapter [SWS-791] #873

gregturn opened this issue Aug 8, 2012 · 24 comments
Assignees
Milestone

Comments

@gregturn
Copy link
Member

@gregturn gregturn commented Aug 8, 2012

Arvind Patel opened SWS-791 and commented

I have a static-wsdl defined and in wsdl I have inported a schema with schemaLocation attribute.
When I deploy this to weblogic spring doesn't transform the schemaLocation to where it's deployed.

I do have transformWsdlLocations set to true in my web.xml.

The location in the service section of WSDL does get transformed but schemaLocation is not.

Please help.

Thanks,


Affects: 2.0 GA

Issue Links:

  • #777 XsdSchemaHandlerAdapter does not transform schema locations

Referenced from: commits 13f537f

@gregturn
Copy link
Member Author

@gregturn gregturn commented Aug 9, 2012

Arjen Poutsma commented

The WsdlDefinitionHandlerAdapter currently does not support transforming the schemaLocation. I will consider this issue a feature request for that functionality.

@gregturn
Copy link
Member Author

@gregturn gregturn commented Aug 14, 2012

Arjen Poutsma commented

Fixed. WsdlDefinitionAdapter now has a transformSchemaLocations property, which defaults to false. When set to true, it will transform all schemaLocation attributes found in the WSDL, just like transformLocations does.

@gregturn
Copy link
Member Author

@gregturn gregturn commented Aug 21, 2012

Arvind Patel commented

Hi,
Can you please point me the url where I can download the new version 2.1.1 ?

Thanks,
Arvind

@gregturn
Copy link
Member Author

@gregturn gregturn commented Aug 22, 2012

Arjen Poutsma commented

@Arvind, it's not out yet. Keep an eye on the forum to see when it's released.

@gregturn
Copy link
Member Author

@gregturn gregturn commented Sep 4, 2012

Arvind Patel commented

Hi Arjen,

We are running behind deadlines and was wondering if the new version 2.1.1 out yet.

let me know.

Regards,
Arvind

@gregturn
Copy link
Member Author

@gregturn gregturn commented Sep 18, 2012

Arjen Poutsma commented

It's out now.

@gregturn
Copy link
Member Author

@gregturn gregturn commented Sep 24, 2012

Arvind Patel commented

Hi Arjen,

I tried this and it does replace the schemaLocation in the wsdl but It does NOT replace schemaLocation in XSDs.

For example I have Employee.wsdl which has the following line

<xsd:import namespace="http://www.localhost/schema/EmployeeService" schemaLocation="http://localhost/employee/EmployeeService.xsd"/>

This does get replaced with actual server name :

But Now if my EmployeeService.xsd has the following line , it does not replace with the actual server name;

<xsd:import namespace="http://www.localhost/schema/Employee" schemaLocation="http://localhost/employee/Employee.xsd"/>

Please advise.

Thanks,
Arvind.

@gregturn
Copy link
Member Author

@gregturn gregturn commented Sep 25, 2012

Arjen Poutsma commented

Indeed: I only changed the code in the WsdlDefinitionHandlerAdapter (as requested), but I should also change the XsdSchemaHandlerAdapter, which serves the XSD files.

Will fix for 2.1.2.

@gregturn
Copy link
Member Author

@gregturn gregturn commented Nov 2, 2012

Arvind Patel commented

Let me know if this issue is resolved and if it's part of nightly build so that I can try.

Thanks,

@gregturn
Copy link
Member Author

@gregturn gregturn commented Nov 5, 2012

Arjen Poutsma commented

If you are talking about #777: yes that should be in the nightly build.

@gregturn
Copy link
Member Author

@gregturn gregturn commented Nov 5, 2012

Arvind Patel commented

It's related to #777 , but it does not completely solve the problem.

Please refer to my comment above from 24/Sep/12 8:07 AM and your reply on 25/Sep/12 5:21 AM

Please let me know if this is part of the nightly build.

Thanks,
Arvind

@gregturn
Copy link
Member Author

@gregturn gregturn commented Nov 6, 2012

Arjen Poutsma commented

It should be part of the build. Let me know if it doesn't work for you.

@gregturn
Copy link
Member Author

@gregturn gregturn commented Nov 6, 2012

Arvind Patel commented

I tried the new build but it does not replace the schema locations in XSD.

Is there a special property that I need to set ?

Thanks,

@gregturn
Copy link
Member Author

@gregturn gregturn commented Nov 7, 2012

Arjen Poutsma commented

I made some additional changes, and now you should be able to enable XSD transformations by setting the transformSchemaLocations property to true. Please use a recent snapshot to try this out (as of tonight).

@gregturn
Copy link
Member Author

@gregturn gregturn commented Nov 7, 2012

Arvind Patel commented

I tried the new version and It still does not work.
The configuration I have is

<bean id="wsdlDefinitionHandlerAdapter" class="org.springframework.ws.transport.http.WsdlDefinitionHandlerAdapter">
<property name="transformSchemaLocations" value="true"/>
</bean>

Let me know if I am missing anything.

Thanks,

@gregturn
Copy link
Member Author

@gregturn gregturn commented Nov 9, 2012

Arjen Poutsma commented

Ok, I'll reopen this issue and take a look at it again.

@gregturn
Copy link
Member Author

@gregturn gregturn commented Nov 19, 2012

Arvind Patel commented

I took nightly built jars and it worked now.

Thanks a lot for working on this.

Let me know when you include this in a new release.

Thanks again !

@gregturn
Copy link
Member Author

@gregturn gregturn commented Nov 26, 2012

Arjen Poutsma commented

Good to hear!

@gregturn
Copy link
Member Author

@gregturn gregturn commented Nov 29, 2012

Arvind Patel commented

Arjen,

Can you give me the timeframe when the new version (2.1.2)is going to be relased ?

Thanks,

@gregturn
Copy link
Member Author

@gregturn gregturn commented Nov 30, 2012

Arjen Poutsma commented

I hope to do it before the end of the year. There are still 8 issues to resolve, and I cannot yet tell how much time they will take.

@gregturn
Copy link
Member Author

@gregturn gregturn commented Dec 4, 2012

Arvind Patel commented

When I go to download page and click on link to download it gives me error and takes me to the page with following contents

<Error>
<Code>AccessDenied</Code>
<Message>Access Denied</Message>
<RequestId>21BD3C619D51FCBF</RequestId>
<HostId>
+WultixhXrHOmYTTzoYS3xdZbOohDoJbLeNrCMvHL6zDlJbZiQIeForthbcnYNd1
</HostId>
</Error>

@gregturn
Copy link
Member Author

@gregturn gregturn commented Dec 4, 2012

Arjen Poutsma commented

Yes, I screwed up with the rights. Try again, this should be fixed now.

@gregturn
Copy link
Member Author

@gregturn gregturn commented Dec 4, 2012

Arvind Patel commented

It worked, thank you,

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants
You can’t perform that action at this time.