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

Warning: SimpleXMLElement::__construct() .../Feed.php on line 172 #976

Closed
leganzaro opened this issue May 14, 2020 · 3 comments · Fixed by #994
Closed

Warning: SimpleXMLElement::__construct() .../Feed.php on line 172 #976

leganzaro opened this issue May 14, 2020 · 3 comments · Fixed by #994
Labels
bug Component: AdminNotification Relates to Mage_AdminNotification

Comments

@leganzaro
Copy link

Last version form LTS 1.9.4.x and php 7.2.30.

We've encountered the issue when we tried to use "Flush Magento Cache" from backend. The error occurs in our logs as a warning and we are not sure how to solve this.

Logged Error 	Warning: SimpleXMLElement::__construct(): Entity: line 6: parser error : Opening and ending tag mismatch: hr line 5 and body in /***/Mage/AdminNotification/Model/Feed.php on line 172
Logged Error 	Warning: SimpleXMLElement::__construct(): </body> in /***/Mage/AdminNotification/Model/Feed.php on line 172
Logged Error 	Warning: SimpleXMLElement::__construct(): ^ in /***/Mage/AdminNotification/Model/Feed.php on line 172
Logged Error 	Warning: SimpleXMLElement::__construct(): Entity: line 7: parser error : Opening and ending tag mismatch: body line 3 and html in /***/Mage/AdminNotification/Model/Feed.php on line 172
Logged Error 	Warning: SimpleXMLElement::__construct(): Entity: line 7: parser error : Premature end of data in tag html line 1 in /***/Mage/AdminNotification/Model/Feed.php on line 172
@tmotyl
Copy link
Contributor

tmotyl commented May 14, 2020

the data returned by url is

<html>
<head><title>301 Moved Permanently</title></head>
<body>
<center><h1>301 Moved Permanently</h1></center>
<hr><center>nginx</center>
</body>
</html>

the called url is: ‌https://openmage.github.io/Web_Notifications/notifications_v1.rss

which redirects to http version:

HTTP/2 301
server: GitHub.com
content-type: text/html
location: http://www.openmage.org/Web_Notifications/notifications_v1.rss

@leganzaro
Copy link
Author

Thank you @tmotyl.

Based on your response, the solution in our case is to copy the core file "../AdminNotification/etc/config.xml" to local and remove the feed url.

@colinmollenhour
Copy link
Member

@Flyingmana Any reason not to enable Enforce HTTPS?
image

@Flyingmana Flyingmana mentioned this issue May 20, 2020
Merged
@sreichel sreichel added bug fixed Component: AdminNotification Relates to Mage_AdminNotification labels Jun 19, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Component: AdminNotification Relates to Mage_AdminNotification
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

update adminnotification url to www.openmage.org OpenMage/magento-lts
4 participants
@colinmollenhour @tmotyl @sreichel @leganzaro