Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 327849 - ServerPreferenceModifyListener producing "[Fatal Error]..." log output, but there are no actual perceived failures.
Summary: ServerPreferenceModifyListener producing "[Fatal Error]..." log output, but t...
Status: RESOLVED FIXED
Alias: None
Product: WTP ServerTools
Classification: WebTools
Component: wst.server (show other bugs)
Version: 3.2   Edit
Hardware: PC Windows XP
: P2 normal (vote)
Target Milestone: 3.2.3   Edit
Assignee: Angel Vera CLA
QA Contact: Angel Vera CLA
URL: ORACLE_P3
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-10-15 00:00 EDT by Ian Trimble CLA
Modified: 2017-10-11 16:35 EDT (History)
2 users (show)

See Also:

Attachments
v1.0 (2.02 KB, patch)
2011-01-07 14:45 EST, Angel Vera CLA 		no flags Details | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Ian Trimble CLA 2010-10-15 00:00:56 EDT 
While running JSF Tools JUnit tests (many of the suites), org.eclipse.wst.server.core.internal.ServerPreferenceModifyListener.removeLockedServerRuntimePreference(Preferences) is writing "[Fatal Error] :1:1: Premature end of file." to syserr. However, there are no perceived failures or errors. It's possibly due to an attempted parse of a non-existent file (I debugged only so far). It may be harmless noise, I am not certain.
Comment 1 Angel Vera CLA 2010-10-15 09:52:40 EDT 
This should be a harmless failure.. but I can see how a annoying it could be. This was introduced by bug# 267188
Comment 2 Ian Trimble CLA 2010-10-15 12:07:50 EDT 
It does seem to be harmless - but repeatedly spitting "Fatal Error" to stderr doesn't give anyone a warm, fuzzy feeling.
Comment 3 Angel Vera CLA 2011-01-07 14:45:57 EST 
Created attachment 186305 [details]
v1.0

The following patch will resolve the problem. 

I can simplify the code a little more and avoid creating the InputSource and StringReader, but to avoid regression testing for now (as I there no JUnit that cover this area) I will leave it as is and come back to it later.
Comment 4 Angel Vera CLA 2011-01-07 14:50:19 EST 
changes committed to 32M
Comment 5 Angel Vera CLA 2011-01-07 14:51:38 EST 
changes released to 32M
Comment 6 Angel Vera CLA 2011-01-07 15:07:12 EST 
changes committed to HEAD
Comment 7 Angel Vera CLA 2011-01-07 15:11:26 EST 
changes released to HEAD
Comment 8 Eclipse Genie CLA 2017-10-11 16:35:29 EDT 
New Gerrit change created: https://git.eclipse.org/r/109013