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

Text files being served as 'text/plain' instead of expected 'text' #412

Closed
yaustar opened this issue Jun 6, 2021 · 5 comments · Fixed by playcanvas/engine#3227
Closed
Assignees
Labels
area: publishing bug Something isn't working

Comments

@yaustar
Copy link
Contributor

yaustar commented Jun 6, 2021

Example project: https://playcanvas.com/project/802245/overview/text-served-incorrectly

image

Not 100% sure if this is server config issue or an engine issue.

@yaustar yaustar added area: publishing bug Something isn't working labels Jun 6, 2021
@vkalpias
Copy link

vkalpias commented Jun 8, 2021

This is just a log in the engine - it is not a problem other than an unnecessary warning in this case.

@yaustar
Copy link
Contributor Author

yaustar commented Jun 8, 2021

Should the warning be wrapped in #debug builds in that case?

@vfernandestoptal
Copy link

It would be great for us if it didn't show in production builds, since we are sending it to our logs and it becomes noise

@yaustar
Copy link
Contributor Author

yaustar commented Jun 10, 2021

Created PR playcanvas/engine#3227

@403-Fruit
Copy link

Could this be reconsidered? In some projects it's outputting 50+ warnings for me. Keep in mind that the console is frequently used in debug builds as well.

image

Is there actually any useful information being conveyed to the developer? If not, it would be nice if it could be removed completely. Otherwise it's not a big deal, just a minor annoyance.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: publishing bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants