• Post Reply Bookmark Topic Watch Topic
  • New Topic
programming forums Java Mobile Certification Databases Caching Books Engineering Micro Controllers OS Languages Paradigms IDEs Build Tools Frameworks Application Servers Open Source This Site Careers Other all forums
this forum made possible by our volunteer staff, including ...
Marshals:
  • Campbell Ritchie
  • Liutauras Vilda
  • Bear Bibeault
  • Jeanne Boyarsky
  • paul wheaton
Sheriffs:
  • Junilu Lacar
  • Paul Clapham
  • Knute Snortum
Saloon Keepers:
  • Stephan van Hulst
  • Ron McLeod
  • Tim Moores
  • salvin francis
  • Carey Brown
Bartenders:
  • Tim Holloway
  • Frits Walraven
  • Vijitha Kumara

Problem performing change password application using jsp  RSS feed

 
Ranch Hand
Posts: 38
  • Mark post as helpful
  • send pies
  • Quote
  • Report post to moderator


This is the Javascript and designing page
the jsp connection file with the database is:


Database consists of only id and oldpassword fields.
The problem here the update query in not firing and there is no change in the database.Please help
 
Saloon Keeper
Posts: 5329
143
  • Mark post as helpful
  • send pies
  • Quote
  • Report post to moderator
Several points, in no particular order:

Java code belongs in a backing bean or a servlet, not jn JSPs (which should only be used to generate the view).

The code is wide open to SQL injection attacks; start using PreparedStatement. You seem to have started doing that, but the parameter isn't added correctly, and it isn't actually being used.

The passwords seem to be stored in cleartext, which is another big no-no; they should be hashed using an algorithm such as SHA-2. That way they can not ever be accessed as cleartext.

As to debugging, I suggest to print out the full stack trace of exceptions, not just what line 43 does.
 
VarunS Singh
Ranch Hand
Posts: 38
  • Mark post as helpful
  • send pies
  • Quote
  • Report post to moderator
thanks for the help
 
Marshal
Posts: 67255
170
IntelliJ IDE Java jQuery Mac Mac OS X
  • Likes 1
  • Mark post as helpful
  • send pies
  • Quote
  • Report post to moderator

VarunS Singh wrote:This is the Javascript ...


That is not JavaScript. It's just Java code that is inappropriately (as Tim pointed out) placed in the JSP.

JavaScript is a completely different language.
 
Rancher
Posts: 3985
47
  • Likes 1
  • Mark post as helpful
  • send pies
  • Quote
  • Report post to moderator
I think that was referring to the code block above it, and not the one below.
 
Bear Bibeault
Marshal
Posts: 67255
170
IntelliJ IDE Java jQuery Mac Mac OS X
  • Likes 1
  • Mark post as helpful
  • send pies
  • Quote
  • Report post to moderator

Dave Tolls wrote:I think that was referring to the code block above it, and not the one below.


Could be. Best to make sure.
 
Anderson gave himself the promotion. So I gave myself this tiny ad:
global solutions you can do at home or in your backyard
https://www.kickstarter.com/projects/paulwheaton/better-world-boo
  • Post Reply Bookmark Topic Watch Topic
  • New Topic
Boost this thread!