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

FBTraceManager.loadConfigurationFromFile [JDBC493] #531

Closed
firebird-issue-importer opened this issue May 9, 2017 · 6 comments
Closed

FBTraceManager.loadConfigurationFromFile [JDBC493] #531

firebird-issue-importer opened this issue May 9, 2017 · 6 comments

Comments

@firebird-issue-importer
Copy link

@firebird-issue-importer firebird-issue-importer commented May 9, 2017

Submitted by: Freddy Ertl (freddyertl)

Method loadConfigurationFromFile strips line separator which leads to a parsing error in Firebird. There is a sb.append(System.getProperty("line.separator")); missing in the while loop.

Commits: 291fa0c 600535d a147e94 696530f

@firebird-issue-importer
Copy link
Author

@firebird-issue-importer firebird-issue-importer commented May 10, 2017

Modified by: @mrotteveel

Fix Version: Jaybird 3.0.1 [ 10831 ]

Fix Version: Jaybird 2.2.14 [ 10820 ]

@firebird-issue-importer
Copy link
Author

@firebird-issue-importer firebird-issue-importer commented May 10, 2017

Modified by: @mrotteveel

Fix Version: Jaybird 3.1 [ 10441 ]

@firebird-issue-importer
Copy link
Author

@firebird-issue-importer firebird-issue-importer commented May 10, 2017

Commented by: @mrotteveel

Fixed in 2.2.14, 3.0.1 and 3.1

I have deprecated this method for removal in Jaybird 4 as this can be handled using standard Java functionality as well (and better, as this method uses the default character set).

Thanks for reporting!

@firebird-issue-importer
Copy link
Author

@firebird-issue-importer firebird-issue-importer commented May 10, 2017

Modified by: @mrotteveel

status: Open [ 1 ] => Resolved [ 5 ]

resolution: Fixed [ 1 ]

@firebird-issue-importer
Copy link
Author

@firebird-issue-importer firebird-issue-importer commented Jun 5, 2017

Commented by: @mrotteveel

Released with Jaybird 3.0.1 : https://github.com/FirebirdSQL/jaybird/releases/tag/v3.0.1

@firebird-issue-importer
Copy link
Author

@firebird-issue-importer firebird-issue-importer commented Jun 5, 2017

Modified by: @mrotteveel

status: Resolved [ 5 ] => Closed [ 6 ]

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment