PHPC-1893: Implement ServerOpeningEvent class

Author: tanlisu

config.m4

@@ -173,6 +173,7 @@ if test "$PHP_MONGODB" != "no"; then
     src/MongoDB/Monitoring/CommandSucceededEvent.c \
     src/MongoDB/Monitoring/SDAMSubscriber.c \
     src/MongoDB/Monitoring/Subscriber.c \
+    src/MongoDB/Monitoring/ServerOpeningEvent.c \
     src/MongoDB/Monitoring/TopologyChangedEvent.c \
     src/MongoDB/Monitoring/TopologyOpeningEvent.c \
     src/MongoDB/Monitoring/functions.c \

config.w32

@@ -122,7 +122,7 @@ if (PHP_MONGODB != "no") {
   MONGODB_ADD_SOURCES("/src/BSON", "Binary.c BinaryInterface.c DBPointer.c Decimal128.c Decimal128Interface.c Int64.c Javascript.c JavascriptInterface.c MaxKey.c MaxKeyInterface.c MinKey.c MinKeyInterface.c ObjectId.c ObjectIdInterface.c Persistable.c Regex.c RegexInterface.c Serializable.c Symbol.c Timestamp.c TimestampInterface.c Type.c Undefined.c Unserializable.c UTCDateTime.c UTCDateTimeInterface.c functions.c");
   MONGODB_ADD_SOURCES("/src/MongoDB", "BulkWrite.c ClientEncryption.c Command.c Cursor.c CursorId.c CursorInterface.c Manager.c Query.c ReadConcern.c ReadPreference.c Server.c ServerApi.c ServerDescription.c Session.c TopologyDescription.c WriteConcern.c WriteConcernError.c WriteError.c WriteResult.c");
   MONGODB_ADD_SOURCES("/src/MongoDB/Exception", "AuthenticationException.c BulkWriteException.c CommandException.c ConnectionException.c ConnectionTimeoutException.c EncryptionException.c Exception.c ExecutionTimeoutException.c InvalidArgumentException.c LogicException.c RuntimeException.c ServerException.c SSLConnectionException.c UnexpectedValueException.c WriteException.c");
-  MONGODB_ADD_SOURCES("/src/MongoDB/Monitoring", "CommandFailedEvent.c CommandStartedEvent.c CommandSubscriber.c CommandSucceededEvent.c SDAMSubscriber.c Subscriber.c TopologyChangedEvent.c TopologyOpeningEvent.c functions.c");
+  MONGODB_ADD_SOURCES("/src/MongoDB/Monitoring", "CommandFailedEvent.c CommandStartedEvent.c CommandSubscriber.c CommandSucceededEvent.c SDAMSubscriber.c Subscriber.c ServerOpeningEvent.c TopologyChangedEvent.c TopologyOpeningEvent.c functions.c");
   MONGODB_ADD_SOURCES("/src/libmongoc/src/common", PHP_MONGODB_COMMON_SOURCES);
   MONGODB_ADD_SOURCES("/src/libmongoc/src/libbson/src/bson", PHP_MONGODB_BSON_SOURCES);
   MONGODB_ADD_SOURCES("/src/libmongoc/src/libbson/src/jsonsl", PHP_MONGODB_JSONSL_SOURCES);

php_phongo.c

@@ -3857,6 +3857,7 @@ PHP_MINIT_FUNCTION(mongodb)
 	php_phongo_commandstartedevent_init_ce(INIT_FUNC_ARGS_PASSTHRU);
 	php_phongo_commandsucceededevent_init_ce(INIT_FUNC_ARGS_PASSTHRU);
 	php_phongo_sdamsubscriber_init_ce(INIT_FUNC_ARGS_PASSTHRU);
+	php_phongo_serveropeningevent_init_ce(INIT_FUNC_ARGS_PASSTHRU);
 	php_phongo_topologychangedevent_init_ce(INIT_FUNC_ARGS_PASSTHRU);
 	php_phongo_topologyopeningevent_init_ce(INIT_FUNC_ARGS_PASSTHRU);
 

php_phongo_classes.h

@@ -159,6 +159,10 @@ static inline php_phongo_commandsucceededevent_t* php_commandsucceededevent_fetc
 {
 	return (php_phongo_commandsucceededevent_t*) ((char*) obj - XtOffsetOf(php_phongo_commandsucceededevent_t, std));
 }
+static inline php_phongo_serveropeningevent_t* php_serveropeningevent_fetch_object(zend_object* obj)
+{
+	return (php_phongo_serveropeningevent_t*) ((char*) obj - XtOffsetOf(php_phongo_serveropeningevent_t, std));
+}
 static inline php_phongo_topologychangedevent_t* php_topologychangedevent_fetch_object(zend_object* obj)
 {
 	return (php_phongo_topologychangedevent_t*) ((char*) obj - XtOffsetOf(php_phongo_topologychangedevent_t, std));
@@ -202,6 +206,7 @@ static inline php_phongo_topologyopeningevent_t* php_topologyopeningevent_fetch_
 #define Z_COMMANDFAILEDEVENT_OBJ_P(zv) (php_commandfailedevent_fetch_object(Z_OBJ_P(zv)))
 #define Z_COMMANDSTARTEDEVENT_OBJ_P(zv) (php_commandstartedevent_fetch_object(Z_OBJ_P(zv)))
 #define Z_COMMANDSUCCEEDEDEVENT_OBJ_P(zv) (php_commandsucceededevent_fetch_object(Z_OBJ_P(zv)))
+#define Z_SERVEROPENINGEVENT_OBJ_P(zv) (php_serveropeningevent_fetch_object(Z_OBJ_P(zv)))
 #define Z_TOPOLOGYCHANGEDEVENT_OBJ_P(zv) (php_topologychangedevent_fetch_object(Z_OBJ_P(zv)))
 #define Z_TOPOLOGYOPENINGEVENT_OBJ_P(zv) (php_topologyopeningevent_fetch_object(Z_OBJ_P(zv)))
 
@@ -239,6 +244,7 @@ static inline php_phongo_topologyopeningevent_t* php_topologyopeningevent_fetch_
 #define Z_OBJ_COMMANDFAILEDEVENT(zo) (php_commandfailedevent_fetch_object(zo))
 #define Z_OBJ_COMMANDSTARTEDEVENT(zo) (php_commandstartedevent_fetch_object(zo))
 #define Z_OBJ_COMMANDSUCCEEDEDEVENT(zo) (php_commandsucceededevent_fetch_object(zo))
+#define Z_OBJ_SERVEROPENINGEVENT(zo) (php_serveropeningevent_fetch_object(zo))
 #define Z_OBJ_TOPOLOGYCHANGEDEVENT(zo) (php_topologychangedevent_fetch_object(zo))
 #define Z_OBJ_TOPOLOGYOPENINGEVENT(zo) (php_topologyopeningevent_fetch_object(zo))
 
@@ -313,6 +319,7 @@ extern zend_class_entry* php_phongo_commandsubscriber_ce;
 extern zend_class_entry* php_phongo_commandsucceededevent_ce;
 extern zend_class_entry* php_phongo_sdamsubscriber_ce;
 extern zend_class_entry* php_phongo_subscriber_ce;
+extern zend_class_entry* php_phongo_serveropeningevent_ce;
 extern zend_class_entry* php_phongo_topologychangedevent_ce;
 extern zend_class_entry* php_phongo_topologyopeningevent_ce;
 
@@ -387,6 +394,7 @@ extern void php_phongo_commandsubscriber_init_ce(INIT_FUNC_ARGS);
 extern void php_phongo_commandsucceededevent_init_ce(INIT_FUNC_ARGS);
 extern void php_phongo_sdamsubscriber_init_ce(INIT_FUNC_ARGS);
 extern void php_phongo_subscriber_init_ce(INIT_FUNC_ARGS);
+extern void php_phongo_serveropeningevent_init_ce(INIT_FUNC_ARGS);
 extern void php_phongo_topologychangedevent_init_ce(INIT_FUNC_ARGS);
 extern void php_phongo_topologyopeningevent_init_ce(INIT_FUNC_ARGS);
 

php_phongo_structs.h

@@ -284,6 +284,12 @@ typedef struct {
 	zend_object std;
 } php_phongo_commandsucceededevent_t;
 
+typedef struct {
+	bson_oid_t                topology_id;
+	const mongoc_host_list_t* host;
+	zend_object               std;
+} php_phongo_serveropeningevent_t;
+
 typedef struct {
 	bson_oid_t                     topology_id;
 	mongoc_topology_description_t* new_topology_description;

src/MongoDB/Monitoring/ServerOpeningEvent.c

@@ -0,0 +1,141 @@
+/*
+ * Copyright 2021-present MongoDB, Inc.
+ *
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ *   http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
+#include <php.h>
+#include <Zend/zend_interfaces.h>
+
+#ifdef HAVE_CONFIG_H
+#include "config.h"
+#endif
+
+#include "phongo_compat.h"
+#include "php_phongo.h"
+
+zend_class_entry* php_phongo_serveropeningevent_ce;
+
+/* {{{ proto string ServerOpeningEvent::getHost()
+   Returns this event's host */
+static PHP_METHOD(ServerOpeningEvent, getHost)
+{
+	php_phongo_serveropeningevent_t* intern = Z_SERVEROPENINGEVENT_OBJ_P(getThis());
+
+	PHONGO_PARSE_PARAMETERS_NONE();
+
+	RETVAL_STRING((intern->host)->host);
+} /* }}} */
+
+/* {{{ proto MongoDB\BSON\ObjectId ServerOpeningEvent::getTopologyId()
+   Returns this event's topology id */
+static PHP_METHOD(ServerOpeningEvent, getTopologyId)
+{
+	php_phongo_objectid_t*           topology_id;
+	php_phongo_serveropeningevent_t* intern = Z_SERVEROPENINGEVENT_OBJ_P(getThis());
+
+	PHONGO_PARSE_PARAMETERS_NONE();
+
+	object_init_ex(return_value, php_phongo_objectid_ce);
+
+	topology_id = Z_OBJECTID_OBJ_P(return_value);
+	bson_oid_to_string(&intern->topology_id, topology_id->oid);
+	topology_id->initialized = true;
+} /* }}} */
+
+/* {{{ MongoDB\Driver\Monitoring\ServerOpeningEvent function entries */
+ZEND_BEGIN_ARG_INFO_EX(ai_ServerOpeningEvent_void, 0, 0, 0)
+ZEND_END_ARG_INFO()
+
+static zend_function_entry php_phongo_serveropeningevent_me[] = {
+	/* clang-format off */
+	ZEND_NAMED_ME(__construct, PHP_FN(MongoDB_disabled___construct), ai_ServerOpeningEvent_void, ZEND_ACC_PRIVATE | ZEND_ACC_FINAL)
+	PHP_ME(ServerOpeningEvent, getHost, ai_ServerOpeningEvent_void, ZEND_ACC_PUBLIC | ZEND_ACC_FINAL)
+	PHP_ME(ServerOpeningEvent, getTopologyId, ai_ServerOpeningEvent_void, ZEND_ACC_PUBLIC | ZEND_ACC_FINAL)
+	ZEND_NAMED_ME(__wakeup, PHP_FN(MongoDB_disabled___wakeup), ai_ServerOpeningEvent_void, ZEND_ACC_PUBLIC | ZEND_ACC_FINAL)
+	PHP_FE_END
+	/* clang-format on */
+};
+/* }}} */
+
+/* {{{ MongoDB\Driver\Monitoring\ServerOpeningEvent object handlers */
+static zend_object_handlers php_phongo_handler_serveropeningevent;
+
+static void php_phongo_serveropeningevent_free_object(zend_object* object) /* {{{ */
+{
+	php_phongo_serveropeningevent_t* intern = Z_OBJ_SERVEROPENINGEVENT(object);
+
+	zend_object_std_dtor(&intern->std);
+} /* }}} */
+
+static zend_object* php_phongo_serveropeningevent_create_object(zend_class_entry* class_type) /* {{{ */
+{
+	php_phongo_serveropeningevent_t* intern = NULL;
+
+	intern = PHONGO_ALLOC_OBJECT_T(php_phongo_serveropeningevent_t, class_type);
+
+	zend_object_std_init(&intern->std, class_type);
+	object_properties_init(&intern->std, class_type);
+
+	intern->std.handlers = &php_phongo_handler_serveropeningevent;
+
+	return &intern->std;
+} /* }}} */
+
+static HashTable* php_phongo_serveropeningevent_get_debug_info(phongo_compat_object_handler_type* object, int* is_temp) /* {{{ */
+{
+	php_phongo_serveropeningevent_t* intern;
+	zval                             retval = ZVAL_STATIC_INIT;
+	char                             topology_id[25];
+
+	intern   = Z_OBJ_SERVEROPENINGEVENT(PHONGO_COMPAT_GET_OBJ(object));
+	*is_temp = 1;
+	array_init_size(&retval, 2);
+
+	ADD_ASSOC_STRING(&retval, "host", (intern->host)->host);
+
+	bson_oid_to_string(&intern->topology_id, topology_id);
+	ADD_ASSOC_STRING(&retval, "topologyId", topology_id);
+
+	return Z_ARRVAL(retval);
+} /* }}} */
+/* }}} */
+
+void php_phongo_serveropeningevent_init_ce(INIT_FUNC_ARGS) /* {{{ */
+{
+	zend_class_entry ce;
+	(void) type;
+	(void) module_number;
+
+	INIT_NS_CLASS_ENTRY(ce, "MongoDB\\Driver\\Monitoring", "ServerOpeningEvent", php_phongo_serveropeningevent_me);
+	php_phongo_serveropeningevent_ce                = zend_register_internal_class(&ce);
+	php_phongo_serveropeningevent_ce->create_object = php_phongo_serveropeningevent_create_object;
+	PHONGO_CE_FINAL(php_phongo_serveropeningevent_ce);
+	PHONGO_CE_DISABLE_SERIALIZATION(php_phongo_serveropeningevent_ce);
+
+	memcpy(&php_phongo_handler_serveropeningevent, phongo_get_std_object_handlers(), sizeof(zend_object_handlers));
+	php_phongo_handler_serveropeningevent.get_debug_info = php_phongo_serveropeningevent_get_debug_info;
+	php_phongo_handler_serveropeningevent.free_obj       = php_phongo_serveropeningevent_free_object;
+	php_phongo_handler_serveropeningevent.offset         = XtOffsetOf(php_phongo_serveropeningevent_t, std);
+
+	return;
+} /* }}} */
+
+/*
+ * Local variables:
+ * tab-width: 4
+ * c-basic-offset: 4
+ * End:
+ * vim600: noet sw=4 ts=4 fdm=marker
+ * vim<600: noet sw=4 ts=4
+ */

src/phongo_apm.c

@@ -262,6 +262,35 @@ static void phongo_apm_command_failed(const mongoc_apm_command_failed_t* event)
 	FREE_HASHTABLE(subscribers);
 }
 
+static void phongo_apm_server_opening(const mongoc_apm_server_opening_t* event)
+{
+	mongoc_client_t*                 client;
+	HashTable*                       subscribers;
+	php_phongo_serveropeningevent_t* p_event;
+	zval                             z_event;
+
+	client      = mongoc_apm_server_opening_get_context(event);
+	subscribers = phongo_apm_get_subscribers_to_notify(php_phongo_sdamsubscriber_ce, client);
+
+	/* Return early if there are no APM subscribers to notify */
+	if (zend_hash_num_elements(subscribers) == 0) {
+		goto cleanup;
+	}
+
+	object_init_ex(&z_event, php_phongo_serveropeningevent_ce);
+	p_event = Z_SERVEROPENINGEVENT_OBJ_P(&z_event);
+
+	mongoc_apm_server_opening_get_topology_id(event, &p_event->topology_id);
+	p_event->host = mongoc_apm_server_opening_get_host(event);
+
+	phongo_apm_dispatch_event(subscribers, "serverOpening", &z_event);
+	zval_ptr_dtor(&z_event);
+
+cleanup:
+	zend_hash_destroy(subscribers);
+	FREE_HASHTABLE(subscribers);
+}
+
 static void phongo_apm_topology_changed(const mongoc_apm_topology_changed_t* event)
 {
 	mongoc_client_t*                   client;
@@ -311,7 +340,6 @@ static void phongo_apm_topology_opening(const mongoc_apm_topology_opening_t* eve
 	p_event = Z_TOPOLOGYOPENINGEVENT_OBJ_P(&z_event);
 
 	mongoc_apm_topology_opening_get_topology_id(event, &p_event->topology_id);
-	;
 
 	phongo_apm_dispatch_event(subscribers, "topologyOpening", &z_event);
 	zval_ptr_dtor(&z_event);
@@ -333,6 +361,7 @@ bool phongo_apm_set_callbacks(mongoc_client_t* client)
 	mongoc_apm_set_command_started_cb(callbacks, phongo_apm_command_started);
 	mongoc_apm_set_command_succeeded_cb(callbacks, phongo_apm_command_succeeded);
 	mongoc_apm_set_command_failed_cb(callbacks, phongo_apm_command_failed);
+	mongoc_apm_set_server_opening_cb(callbacks, phongo_apm_server_opening);
 	mongoc_apm_set_topology_changed_cb(callbacks, phongo_apm_topology_changed);
 	mongoc_apm_set_topology_opening_cb(callbacks, phongo_apm_topology_opening);
 

tests/apm/monitoring-serverOpening-001.phpt

@@ -0,0 +1,44 @@
+--TEST--
+MongoDB\Driver\Monitoring\ServerOpeningEvent
+--SKIPIF--
+<?php require __DIR__ . "/../utils/basic-skipif.inc"; ?>
+<?php skip_if_not_live(); ?>
+--FILE--
+<?php
+require_once __DIR__ . "/../utils/basic.inc";
+
+$m = create_test_manager();
+
+class MySubscriber implements MongoDB\Driver\Monitoring\SDAMSubscriber
+{
+    private $serverOpened = false;
+
+    public function serverOpening(MongoDB\Driver\Monitoring\ServerOpeningEvent $event)
+    {
+        if ($this->serverOpened) {
+            return;
+        }
+
+        $this->serverOpened = true;
+        echo "- getHost() returns a string: ", is_string($event->getHost()) ? 'yes' : 'no', "\n";
+        echo "- getTopologyId() returns an ObjectId: ", ($event->getTopologyId() instanceof MongoDB\BSON\ObjectId) ? 'yes' : 'no', "\n";
+    }
+    
+    public function topologyChanged(MongoDB\Driver\Monitoring\TopologyChangedEvent $event) {}
+
+    public function topologyOpening(MongoDB\Driver\Monitoring\TopologyOpeningEvent $event) {}
+}
+
+$subscriber = new MySubscriber;
+MongoDB\Driver\Monitoring\addSubscriber($subscriber);
+
+$command = new MongoDB\Driver\Command(['ping' => 1]);
+$m->executeCommand(DATABASE_NAME, $command);
+
+?>
+===DONE===
+<?php exit(0); ?>
+--EXPECT--
+- getHost() returns a string: yes
+- getTopologyId() returns an ObjectId: yes
+===DONE===

tests/apm/monitoring-topologyChanged-001.phpt

@@ -13,6 +13,8 @@ class MySubscriber implements MongoDB\Driver\Monitoring\SDAMSubscriber
 {
     private $topologyDescription;
 
+    public function serverOpening(MongoDB\Driver\Monitoring\ServerOpeningEvent $event) {}
+
     public function topologyChanged(MongoDB\Driver\Monitoring\TopologyChangedEvent $event)
     {
         if (isset($this->topologyDescription)) {

tests/apm/monitoring-topologyOpening-001.phpt

@@ -11,6 +11,8 @@ $m = create_test_manager();
 
 class MySubscriber implements MongoDB\Driver\Monitoring\SDAMSubscriber
 {
+    public function serverOpening(MongoDB\Driver\Monitoring\ServerOpeningEvent $event) {}
+    
     public function topologyChanged(MongoDB\Driver\Monitoring\TopologyChangedEvent $event) {}
 
     public function topologyOpening(MongoDB\Driver\Monitoring\TopologyOpeningEvent $event)

tests/topologyDescription/topologyDescription-debug-001.phpt

@@ -11,6 +11,8 @@ class TopologyDescriptionProvider implements MongoDB\Driver\Monitoring\SDAMSubsc
 {
     private $topologyDescription;
 
+    public function serverOpening(MongoDB\Driver\Monitoring\ServerOpeningEvent $event) {}
+
     public function topologyChanged(MongoDB\Driver\Monitoring\TopologyChangedEvent $event)
     {
         $this->topologyDescription = $event->getNewDescription();

tests/topologyDescription/topologyDescription-getServers-001.phpt

@@ -11,6 +11,8 @@ class TopologyDescriptionProvider implements MongoDB\Driver\Monitoring\SDAMSubsc
 {
     private $topologyDescription;
 
+    public function serverOpening(MongoDB\Driver\Monitoring\ServerOpeningEvent $event) {}
+
     public function topologyChanged(MongoDB\Driver\Monitoring\TopologyChangedEvent $event)
     {
         $this->topologyDescription = $event->getNewDescription();

tests/topologyDescription/topologyDescription-getType-001.phpt

@@ -11,6 +11,8 @@ class TopologyDescriptionProvider implements MongoDB\Driver\Monitoring\SDAMSubsc
 {
     private $topologyDescription;
 
+    public function serverOpening(MongoDB\Driver\Monitoring\ServerOpeningEvent $event) {}
+
     public function topologyChanged(MongoDB\Driver\Monitoring\TopologyChangedEvent $event)
     {
         $this->topologyDescription = $event->getNewDescription();

tests/topologyDescription/topologyDescription-hasReadableServer-001.phpt

@@ -11,6 +11,8 @@ class TopologyDescriptionProvider implements MongoDB\Driver\Monitoring\SDAMSubsc
 {
     private $topologyDescription;
 
+    public function serverOpening(MongoDB\Driver\Monitoring\ServerOpeningEvent $event) {}
+
     public function topologyChanged(MongoDB\Driver\Monitoring\TopologyChangedEvent $event)
     {
         $this->topologyDescription = $event->getNewDescription();

tests/topologyDescription/topologyDescription-hasReadableServer-002.phpt

@@ -11,6 +11,8 @@ class TopologyDescriptionProvider implements MongoDB\Driver\Monitoring\SDAMSubsc
 {
     private $topologyDescription;
 
+    public function serverOpening(MongoDB\Driver\Monitoring\ServerOpeningEvent $event) {}
+
     public function topologyChanged(MongoDB\Driver\Monitoring\TopologyChangedEvent $event)
     {
         $this->topologyDescription = $event->getNewDescription();

tests/topologyDescription/topologyDescription-hasReadableServer_error-001.phpt

@@ -11,6 +11,8 @@ class TopologyDescriptionProvider implements MongoDB\Driver\Monitoring\SDAMSubsc
 {
     private $topologyDescription;
 
+    public function serverOpening(MongoDB\Driver\Monitoring\ServerOpeningEvent $event) {}
+
     public function topologyChanged(MongoDB\Driver\Monitoring\TopologyChangedEvent $event)
     {
         $this->topologyDescription = $event->getNewDescription();