GitHub Community
UTF-8 BOM character breaks workflow parser #25914
-
|
Hi all, Loving GitHub actions and the new workflow_dispatch functionality! I just wanted to raise this issue which caused me some confusion. I now have a workaround detailed in the link at the bottom, but wanted to know the staff and other users’ thoughts on this. Here is an example patch that will break the parser. It seems to cause unpredictable error messages later in the workflow file (“you have a syntax error at line 15”, etc.) github.comdogboydog/manual-workflow-test/blob/084e6017a349363d88303a39542e921a01b59dbe/failed_parse_patch.patch
This file has been truncated. show original I would link to some failed runs but it seems like they disappeared after I stripped out the character myself and GitHub was able to parse the workflow name. What do you think of this issue? Should the parser strip it out, or is there at least some way the parser can detect it’s presence and give a specific message about it. It seems like it could be safely removed. I know you are not expecting this character but some editors automatically insert it. There is more information here but I figured it is no longer exactly my initial issue from that thread, so I made a new one. Please forgive any frustration I expressed there as I debugged this issue. Thank you |
Beta Was this translation helpful? Give feedback.
Replies: 5 comments
-
|
Hi @dogboydog, I forked your repository and create a new yaml following your patch file. However, i I’m not sure 100% follow, let me know if there are any misunderstanding. The Thanks |
Beta Was this translation helpful? Give feedback.
-
|
Hmm odd. There are screenshots of the error in my other thread but I’ll post them here. Also this pull request from staff from a previous thread involving this issue before I figured out the exact character made the same fix which made my workflow start parsing correctly again: Is it possible GitHub rolled out a fix for this in the last few days? @cschleiden? I can’t get it to happen anymore either, so that’s a good thing. Is there a changelog somewhere that shows bug fixes? I can only find actions to create changelogs by searching. And yeah I know about the input thing. If you’re talking about test_manual2.yaml, that is an example from my other topic that demonstrates that the environmental variable is not set. But thank you Edit: I think they might have just rolled out a fix. I’m happy they did! I missed this message in another topic and didn’t realize they were already tracking this issue. So, sorry if this is a duplicate.
|
Beta Was this translation helpful? Give feedback.
-
|
@dogboydog Thanks for the detailed report. I fixed an issue a bit more than a week ago where we ignore the BOM characters. Let me know if you can still repro this somewhere, happy to take another look :slight_smile: |
Beta Was this translation helpful? Give feedback.
-
|
Thanks for replying. I was wrestling with this issue on Sunday the 26th still (as soon as I stripped the BOM out, it starting parsing again), but I just tried to reproduce and I can’t anymore. If you fixed it more than a week ago I’m not sure why I was still seeing it on Sunday, but it seems all good now. Seems totally fixed. Thank you |
Beta Was this translation helpful? Give feedback.
-
|
Good morning dear, you guys are working hard to make the platform friendly keep it up thank you |
Beta Was this translation helpful? Give feedback.
@dogboydog Thanks for the detailed report. I fixed an issue a bit more than a week ago where we ignore the BOM characters. Let me know if you can still repro this somewhere, happy to take another look :slight_smile: