fix eof for get_binary and get_string

[jprochazk]

Fixes #2293 by breaking when encountering an EOF in binary_reader::get_string() and binary_reader::get_binary()

---

Pull request checklist

• Changes are described in the pull request, or an existing issue is referenced.
• The test suite compiles and runs without error.
• Code coverage is 100%. Test cases can be added by editing the test suite.
• The source code is amalgamated; that is, after making changes to the sources in the include/nlohmann directory, run make amalgamate to create the single-header file single_include/nlohmann/json.hpp. The whole process is described here.

[coveralls]

Coverage remained the same at 100.0% when pulling 851315c on jprochazk:develop into 43ab8a2 on nlohmann:develop.

[nlohmann]

Can you please add the example from #2293 (comment) as regression test?

[nlohmann]

Looks good to me.

[nlohmann]

Thanks a lot!

[nlohmann]

---

🔖 Release item

This issue/PR will be part of the next release of the library. This template helps preparing the release notes.

Type

• ✨ New Feature
• 🐛 Bug Fix
• ⚡️ Improvement
• 🔨 Further Change
• 🔥 Deprecated function

Description

• fix eof for get_binary and get_string fix eof for get_binary and get_string #2294 Application stalls indefinitely with message byte size 10 #2293

---