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

Oracle: Make db_name parameter allow net service name or connection identifier along with SID #431

Merged
merged 5 commits into from
Oct 18, 2018

Conversation

ikedas
Copy link
Member

@ikedas ikedas commented Oct 4, 2018

Preceding posts: sympa-users.

Currently, Oracle database driver expects db_name parameter for Oracle system identifier (SID) and db_host for host name, and both of them are mandatory.

This PR may make db_name parameter allow net service name or (easy) connection identifier along with SID, such as:

db_name SALES1
db_name sales-server:1521/sales.us.acme.com

In cases above db_host and db_port parameters are not necessary.

Note: To specify net service name or connection identifier, DBD-Oracle 1.70 or better is required: See CPAN RT #91775 for details. My misunderstanding.

…ction identifier along with Oracle SID.

Note: DBD-Oracle 1.70 or better is required. cf. CPAN RT #91775.
@ikedas ikedas added this to the 6.2.38 milestone Oct 4, 2018
ikedas added a commit to ikedas/sympa-community.github.io that referenced this pull request Oct 16, 2018
@ikedas
Copy link
Member Author

ikedas commented Oct 16, 2018

This PR will be merged.

I thank E. Cartman for review and suggestions.

@ikedas ikedas merged commit b2b4991 into sympa-community:sympa-6.2 Oct 18, 2018
ikedas added a commit to ikedas/sympa-community.github.io that referenced this pull request Oct 22, 2018
@ikedas ikedas deleted the oracle_with_connection_id branch October 30, 2018 06:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant