Skip to content

Remove invisible unicode character to prevent "invalid byte sequence in ...#149

Merged
kbarber merged 1 commit into
puppetlabs:masterfrom
stefanandres:fix-unneeded-unicode
Oct 16, 2014
Merged

Remove invisible unicode character to prevent "invalid byte sequence in ...#149
kbarber merged 1 commit into
puppetlabs:masterfrom
stefanandres:fix-unneeded-unicode

Conversation

@stefanandres

Copy link
Copy Markdown
Contributor

Someone inserted an unicode character (probably by accident) to the jetty_ini.pp file which prevents puppet-run without unicode environment to run the puppet module.
Those scenarios are likely while bootstrapping new server.

The rest of the files looks good.

$ find . -name '*.pp' -exec file {} +
./manifests/master/config.pp:            ASCII English text
./manifests/master/routes.pp:            ASCII English text
./manifests/master/storeconfigs.pp:      ASCII English text
./manifests/master/puppetdb_conf.pp:     ASCII English text
./manifests/master/report_processor.pp:  ASCII English text
./manifests/server/validate_db.pp:       ASCII English text
./manifests/server/database_ini.pp:      ASCII English text
./manifests/server/jetty_ini.pp:         ASCII English text
./manifests/server/validate_read_db.pp:  ASCII text
./manifests/server/read_database_ini.pp: ASCII English text
./manifests/server/firewall.pp:          ASCII text
./manifests/init.pp:                     ASCII C++ program text
./manifests/params.pp:                   ASCII C++ program text
./manifests/server.pp:                   ASCII English text
./manifests/database/postgresql.pp:      ASCII C++ program text

kbarber added a commit that referenced this pull request Oct 16, 2014
Remove invisible unicode character to prevent "invalid byte sequence in ...
@kbarber
kbarber merged commit 90fd571 into puppetlabs:master Oct 16, 2014
@smortex smortex added the bugfix label Apr 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants