This Bugzilla instance is a read-only archive of historic NetBeans bug reports. To report a bug in NetBeans please follow the project's instructions for reporting issues.

Bug 84817 - Incorrect URL listed for Oracle Driver
Summary: Incorrect URL listed for Oracle Driver
Status: RESOLVED WONTFIX
Alias: None
Product: db
Classification: Unclassified
Component: Code (show other bugs)
Version: 5.x
Hardware: All All
: P2 blocker (vote)
Assignee: Andrei Badea
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2006-09-13 17:50 UTC by _ jimdavidson
Modified: 2006-09-14 22:21 UTC (History)
1 user (show)

See Also:
Issue Type: DEFECT
Exception Reporter:

Attachments
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 _ jimdavidson 2006-09-13 17:50:21 UTC 
Add New Driver for Oracle JDBC Driver (ojdbc14.jar).

Right-click and "Connect Using".

New database connection window appears.  Database URL is shown as:

jdbc:oracle:thin:@<HOST>:<PORT>:<SID>

Per Oracle README, it should be:

jdbc:oracle:thin:@<HOST>:<PORT>/<SID>

"/" instead of ":" before SID.
Comment 1 _ jimdavidson 2006-09-13 19:29:16 UTC 
By the way, the other two URL suggestions in the dropdown need the same fix:

jdbc:oracle:oci8:@<SID>
jdbc:oracle:oci:@<SID>
Comment 2 Andrei Badea 2006-09-14 10:45:40 UTC 
Which README? The Javadoc for the OracleDriver class, which is linked from

http://www.oracle.com/technology/software/tech/java/sqlj_jdbc/htdocs/jdbc_10201.html

only has examples like "jdbc:oracle:thin:@myhost:1521:orcl". Same for the OCI
driver.
Comment 3 _ jimdavidson 2006-09-14 17:41:25 UTC 
Hmmm...  Actually, the README file that you cite does include one example:

"For the JDBC Thin Driver, or Server-side Thin Driver:
ods.setURL("jdbc:oracle:thin:@<database>");

where <database> is either a string of the form
//<host>:<port>/<service_name>, or a SQL*net name-value pair,
or a TNSEntryName."

In other words, with the '/'.

But, as you point out the Javadocs list the ':' notation.

This was originally filed as a Creator bug (#6469978) by one of our QE people. 
I checked the README, but did not test it myself.  Now that I test, it appears
that both forms work.

Feel free to change status to appropriate value.
Comment 4 Andrei Badea 2006-09-14 22:21:24 UTC 
You are right, I didn't notice it in the README file. The colon notation doesn't
seem harmful, so closing in the spirit of "don't fix it if it ain't broken".