[CDRIVER-2257] Initial DNS Seedlist discovery Created: 26/Aug/17  Updated: 28/Oct/23  Resolved: 23/Nov/17

Status: Closed
Project: C Driver
Component/s: None
Affects Version/s: None
Fix Version/s: 1.9.0

Type: New Feature Priority: Major - P3
Reporter: A. Jesse Jiryu Davis Assignee: A. Jesse Jiryu Davis
Resolution: Fixed Votes: 0
Labels: None
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified

Issue Links:
Depends
is depended on by CXX-1438 Implement Initial DNS Seedlist discov... Closed
is depended on by DRIVERS-404 Implement Initial DNS Seedlist discov... Closed
is depended on by PHPC-1015 Implement Initial DNS Seedlist discov... Closed
Related
related to CDRIVER-4465 Cannot build libmongoc with SRV suppo... Closed
related to CDRIVER-3043 valid_hostname() should not rejects s... Closed
related to CDRIVER-2344 Warn if multiple TXT records are found Closed
is related to CDRIVER-2335 Syntax errors in TXT records only war... Closed
Epic Link: Cdriver MongoDB 3.6 Support

 Comments   
Comment by Githook User [ 23/Nov/17 ]

Author:

{'name': 'A. Jesse Jiryu Davis', 'username': 'ajdavis', 'email': 'jesse@mongodb.com'}

Message: CDRIVER-2257 security rules for mongodb+srv://

Hostnames returned from an SRV lookup must share the service name's
parent domain, and SSL is turned on with mongodb+srv URIs by default.
At most one TXT record is allowed, only "replicaSet" and "authSource"
may be set from a TXT record.
Branch: master
https://github.com/mongodb/mongo-c-driver/commit/a625b39a2ef868c879b9a6b5eacd2697ddc9a772

Comment by Githook User [ 21/Nov/17 ]

Author:

{'name': 'A. Jesse Jiryu Davis', 'username': 'ajdavis', 'email': 'jesse@mongodb.com'}

Message: CDRIVER-2257 include cmake file CheckSymbolExists
Branch: master
https://github.com/mongodb/mongo-c-driver/commit/bdcc2ca52aaa42744103ac27a5ada37cb2d39a24

Comment by A. Jesse Jiryu Davis [ 17/Nov/17 ]

Only half the spec updates were reverted: non-ASCII records are not tested now, but records with multiple strings are tested. This is still open while we await the latest tests.

Additionally, we need a spec update that will require hosts to be subdomains of the service name.

Comment by Githook User [ 17/Nov/17 ]

Author:

{'name': 'A. Jesse Jiryu Davis', 'username': 'ajdavis', 'email': 'jesse@mongodb.com'}

Message: CDRIVER-2257 update DNS tests
Branch: master
https://github.com/mongodb/mongo-c-driver/commit/b6aa249ba706667f86615ccc39092823de3748e1

Comment by Githook User [ 17/Nov/17 ]

Author:

{'name': 'A. Jesse Jiryu Davis', 'username': 'ajdavis', 'email': 'jesse@mongodb.com'}

Message: CDRIVER-2257 parse multiple strings in TXT record
Branch: master
https://github.com/mongodb/mongo-c-driver/commit/25096cbf217d730bdac8b20db2718a768ae3944d

Comment by A. Jesse Jiryu Davis [ 15/Nov/17 ]

The spec updates were reverted, our implementation of the previous spec is correct.

Comment by A. Jesse Jiryu Davis [ 07/Nov/17 ]

Reopening: once the latest test records are deployed by the Build Team we need to re-sync the tests and update our implementation to pass them. In particular, I didn't understand how multiple strings within a record ought to be handled until I saw the tests for them.

Comment by Githook User [ 29/Oct/17 ]

Author:

{'email': 'jesse@mongodb.com', 'name': 'A. Jesse Jiryu Davis', 'username': 'ajdavis'}

Message: CDRIVER-2257 fix DNS tests
Branch: master
https://github.com/mongodb/mongo-c-driver/commit/849375b8a42c236e0c17a5f88a7f7f2f88b4e696

Comment by Githook User [ 28/Oct/17 ]

Author:

{'email': 'jesse@mongodb.com', 'name': 'A. Jesse Jiryu Davis', 'username': 'ajdavis'}

Message: CDRIVER-2257 parse URI options from TXT records
Branch: master
https://github.com/mongodb/mongo-c-driver/commit/60d4fe2db1f08200c26681513206449fc40be218

Comment by Githook User [ 28/Oct/17 ]

Author:

{'email': 'jesse@mongodb.com', 'name': 'A. Jesse Jiryu Davis', 'username': 'ajdavis'}

Message: CDRIVER-2257 use latest DNS discovery tests
Branch: master
https://github.com/mongodb/mongo-c-driver/commit/84bbacf2cb11fb8a7549825d900c2ef3a8161bf8

Comment by Bernie Hackett [ 24/Oct/17 ]

The spec and tests have been updated and TXT records have been provided by the build team.

Comment by Githook User [ 03/Oct/17 ]

Author:

{'email': 'jesse@mongodb.com', 'name': 'A. Jesse Jiryu Davis', 'username': 'ajdavis'}

Message: CDRIVER-2257 handle null error ptr with SRV error
Branch: master
https://github.com/mongodb/mongo-c-driver/commit/78d2fbc20ef99a0dfe7fcb9082445892373f316f

Comment by A. Jesse Jiryu Davis [ 14/Sep/17 ]

BUILD ticket is closed and unblocks us, Jesse will update the Evergreen test and add Windows and Mac to SRV testing. Then close this and await a spec extension to use TXT records for configuration, in a new CDRIVER ticket.

Comment by Ramon Fernandez Marina [ 12/Sep/17 ]

Author:

{'username': u'ajdavis', 'name': u'A. Jesse Jiryu Davis', 'email': u'jesse@mongodb.com'}

Message:CDRIVER-2257 support mongodb+srv URIs
Branch:master
https://github.com/mongodb/mongo-c-driver/commit/5df0b60d804687c0f6278250c03c5f319cb05c85

Comment by Ramon Fernandez Marina [ 12/Sep/17 ]

Author:

{'username': u'ajdavis', 'name': u'A. Jesse Jiryu Davis', 'email': u'jesse@mongodb.com'}

Message:CDRIVER-2257 enable pooled mongodb+srv test
Branch:master
https://github.com/mongodb/mongo-c-driver/commit/0c296315c98a2f59a6b56cae30857f937ece5539

Comment by A. Jesse Jiryu Davis [ 05/Sep/17 ]

Initial implementation works. Awaiting BUILD-3696 before we can rewrite the test.

Comment by Githook User [ 02/Sep/17 ]

Author:

{'username': 'ajdavis', 'name': 'A. Jesse Jiryu Davis', 'email': 'jesse@mongodb.com'}

Message: Merge remote-tracking branch 'upstream/master'

Comment by Githook User [ 01/Sep/17 ]

Author:

{'username': 'bjori', 'name': 'Hannes Magnusson', 'email': 'bjori@php.net'}

Message: CDRIVER-2257 Fix test syntax
Branch: master
https://github.com/mongodb/mongo-c-driver/commit/2b77bf60e2156bb087d70780d0bade424b03a06b

Comment by Githook User [ 01/Sep/17 ]

Author:

{'username': 'ajdavis', 'name': 'A. Jesse Jiryu Davis', 'email': 'jesse@mongodb.com'}

Message: CDRIVER-2257 remove /Client/srv test
Branch: master
https://github.com/mongodb/mongo-c-driver/commit/465372abd7cf5b3953f5481b366cb05eb9c42aab

Comment by Githook User [ 01/Sep/17 ]

Author:

{'username': 'ajdavis', 'name': 'A. Jesse Jiryu Davis', 'email': 'jesse@mongodb.com'}

Message: CDRIVER-2257 libresolv config check sets all options
Branch: master
https://github.com/mongodb/mongo-c-driver/commit/0c6354d5151d5fb5e9af8b86e91e936f2358ed9f

Comment by Githook User [ 01/Sep/17 ]

Author:

{'username': 'ajdavis', 'name': 'A. Jesse Jiryu Davis', 'email': 'jesse@mongodb.com'}

Message: CDRIVER-2257 util function to retrieve SRV records
Branch: master
https://github.com/mongodb/mongo-c-driver/commit/36ebc2e8620f082e60d3ee59e410443b2b3864f1

Generated at Wed Feb 07 21:14:40 UTC 2024 using Jira 9.7.1#970001-sha1:2222b88b221c4928ef0de3161136cc90c8356a66.