Skip to content

Conversation

@JanPetterMG
Copy link
Contributor

Added more feed types.

Pretty common:

<link rel="alternate" type="application/xml" href="https://example.com/feed/" />
<link rel="alternate" type="application/rdf+xml" href="https://example.com/feed/rdf/" />

Less common, but still used on a decent number of webpages:

<link rel="alternate" type="text/xml" href="https://example.com/feed/" />

Added more feed types.
<link rel="alternate" type="XXX" href="XXX" />
@JanPetterMG
Copy link
Contributor Author

JanPetterMG commented Mar 25, 2017

These Embed\Exceptions\InvalidUrlException: Invalid url (...) issues in the Travis check that failed, is that a recently introduced issue?
I'm receiving lots of InvalidUrlExceptions for no obvious reason, using the latest v3.0.5.
I've just upgraded from v2.7.7 btw...

EDIT: Probably unrelated to issue 200, as the urls named in the exceptions doesn't include UTF-8 characters.

@oscarotero oscarotero merged commit 86f8e86 into php-embed:master Mar 26, 2017
@oscarotero
Copy link
Collaborator

thanks

PD: The issues in the travis test is because changes in the webpages, not in this library. I've fixed them.

@JanPetterMG JanPetterMG deleted the patch-1 branch March 26, 2017 00:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants