A couple of points:
for your SQL try using a PreparedStatement and using the setXXX methods on it.

Greg's point about Patrick O'Reilly was that with your hardcoded update statement you 
don't have any method of escaping single
quotes in the input data.
Also it looks like you are sending a string where the database likely expects a date 
field.

Also the odbc-jdbc driver is not thread safe. If you have two people registering near 
enough the same time you will have weird hard
to re-produce problems with your servlet.
With your current setup you need to at least modify your InsertIntoDB method so that 
it is synchronized on the connection.

Then you need to find a way to make your servlet re-load the connection if someone 
restarts the access database...

A solution I think tends to work would be to use a ConnectionPool that allows only a 
single connection. You'd get the connection in
your InsertIntoDB method and would no longer need to worry about synchronization. The 
pool should manage re-getting the connection
after an access restart.

HTH

***********************************************************
Brett Knights                             250-338-3509 work
[EMAIL PROTECTED]                 250-334-8309 home
***********************************************************

Reply via email to