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

Use https instead of http for VIES_WSDL_URL #76

Merged
merged 1 commit into from
Apr 17, 2020
Merged

Use https instead of http for VIES_WSDL_URL #76

merged 1 commit into from
Apr 17, 2020

Conversation

lubekpl
Copy link
Contributor

@lubekpl lubekpl commented Apr 17, 2020

Seems that http URL started redirecting to https

❯ curl -v http://ec.europa.eu/taxation_customs/vies/checkVatService.wsdl
*   Trying 2a01:7080:24:100::666:30...
* TCP_NODELAY set
*   Trying 147.67.210.30...
* TCP_NODELAY set
* Connected to ec.europa.eu (2a01:7080:24:100::666:30) port 80 (#0)
> GET /taxation_customs/vies/checkVatService.wsdl HTTP/1.1
> Host: ec.europa.eu
> User-Agent: curl/7.54.0
> Accept: */*
>
< HTTP/1.1 307 Temporary Redirect
< Location: https://ec.europa.eu/taxation_customs/vies/checkVatService.wsdl
< Cache-Control: no-cache
< Pragma: no-cache
< Content-Type: text/html; charset=utf-8
< Proxy-Connection: close
< Connection: close
< Content-Length: 738
<
<HTML><HEAD>
<TITLE>Redirect</TITLE>
</HEAD>
<BODY>
<FONT face="Helvetica">
<big><strong></strong></big><BR>
</FONT>
<blockquote>
<TABLE border=0 cellPadding=1 width="80%">
<TR><TD>
<FONT face="Helvetica">
<big>Redirect (policy_request_redirect)</big>
<BR>
<BR>
</FONT>
</TD></TR>
<TR><TD>
<FONT face="Helvetica">
Click <a href="https:&#x2F;&#x2F;ec.europa.eu&#x2F;taxation_customs&#x2F;vies&#x2F;checkVatService.wsdl">here</a> if you are not automatically redirected.
</FONT>
</TD></TR>
<TR><TD>
<FONT face="Helvetica">

</FONT>
</TD></TR>
<TR><TD>
<FONT face="Helvetica" SIZE=2>
<BR>
For assistance, contact your network support team.
</FONT>
</TD></TR>
</TABLE>
</blockquote>
</FONT>
</BODY></HTML>
* Closing connection 0```

@lubekpl
Copy link
Contributor Author

lubekpl commented Apr 17, 2020

Duplicate of #74

@lubekpl lubekpl closed this Apr 17, 2020
@yolk
Copy link
Owner

yolk commented Apr 17, 2020

This seems more like the proper solution to #74. I am going to reopen it.

@yolk yolk reopened this Apr 17, 2020
@yolk yolk merged commit 15d2fbc into yolk:master Apr 17, 2020
@yolk
Copy link
Owner

yolk commented Apr 17, 2020

At least it will prevent some unnecessary redirects :-)

Thanks a lot!

@lubekpl lubekpl deleted the change_vies_url branch April 21, 2020 16:22
@avschiller avschiller mentioned this pull request Jun 20, 2020
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@lubekpl @yolk