<!-- 
RSS generated by JIRA (9.7.1#970001-sha1:2222b88b221c4928ef0de3161136cc90c8356a66) at Wed Feb 07 21:17:39 UTC 2024

It is possible to restrict the fields that are returned in this document by specifying the 'field' parameter in your request.
For example, to request only the issue key and summary append 'field=key&field=summary' to the URL of your request.
-->
<rss version="0.92" >
<channel>
    <title>MongoDB Jira</title>
    <link>https://jira.mongodb.org</link>
    <description>This file is an XML representation of an issue</description>
    <language>en-us</language>    <build-info>
        <version>9.7.1</version>
        <build-number>970001</build-number>
        <build-date>13-04-2023</build-date>
    </build-info>


<item>
            <title>[CDRIVER-3303] Some options not applied for OP_MSG without sessions</title>
                <link>https://jira.mongodb.org/browse/CDRIVER-3303</link>
                <project id="10030" key="CDRIVER">C Driver</project>
                    <description>&lt;p&gt;mongoc_cmd_parts_assemble assembles options and the base command document into a final command to send.&lt;/p&gt;

&lt;p&gt;&lt;tt&gt;parts-&amp;gt;body&lt;/tt&gt; is the base command, like &lt;p/&gt;
&lt;div id=&quot;syntaxplugin&quot; class=&quot;syntaxplugin&quot; style=&quot;border: 1px dashed #bbb; border-radius: 5px !important; overflow: auto; max-height: 30em;&quot;&gt;
&lt;table cellspacing=&quot;0&quot; cellpadding=&quot;0&quot; border=&quot;0&quot; width=&quot;100%&quot; style=&quot;font-size: 1em; line-height: 1.4em !important; font-weight: normal; font-style: normal; color: black;&quot;&gt;
		&lt;tbody &gt;
				&lt;tr id=&quot;syntaxplugin_code_and_gutter&quot;&gt;
						&lt;td  style=&quot; line-height: 1.4em !important; padding: 0em; vertical-align: top;&quot;&gt;
					&lt;pre style=&quot;font-size: 1em; margin: 0 10px;  margin-top: 10px;   margin-bottom: 10px;  width: auto; padding: 0;&quot;&gt;&lt;span style=&quot;color: black; font-family: &apos;Consolas&apos;, &apos;Bitstream Vera Sans Mono&apos;, &apos;Courier New&apos;, Courier, monospace !important;&quot;&gt;{insert: &quot;coll&quot;}&lt;/span&gt;&lt;/pre&gt;
			&lt;/td&gt;
		&lt;/tr&gt;
			&lt;/tbody&gt;
&lt;/table&gt;
&lt;/div&gt;
&lt;p/&gt;&lt;br/&gt;
&lt;tt&gt;parts-&amp;gt;assembled_body&lt;/tt&gt; is the base command with options set, like:&lt;/p&gt;
&lt;p/&gt;
&lt;div id=&quot;syntaxplugin&quot; class=&quot;syntaxplugin&quot; style=&quot;border: 1px dashed #bbb; border-radius: 5px !important; overflow: auto; max-height: 30em;&quot;&gt;
&lt;table cellspacing=&quot;0&quot; cellpadding=&quot;0&quot; border=&quot;0&quot; width=&quot;100%&quot; style=&quot;font-size: 1em; line-height: 1.4em !important; font-weight: normal; font-style: normal; color: black;&quot;&gt;
		&lt;tbody &gt;
				&lt;tr id=&quot;syntaxplugin_code_and_gutter&quot;&gt;
						&lt;td  style=&quot; line-height: 1.4em !important; padding: 0em; vertical-align: top;&quot;&gt;
					&lt;pre style=&quot;font-size: 1em; margin: 0 10px;  margin-top: 10px;   margin-bottom: 10px;  width: auto; padding: 0;&quot;&gt;&lt;span style=&quot;color: black; font-family: &apos;Consolas&apos;, &apos;Bitstream Vera Sans Mono&apos;, &apos;Courier New&apos;, Courier, monospace !important;&quot;&gt;{insert: &quot;coll&quot;, &quot;readConcern&quot;: { &quot;level&quot;: &quot;majority&quot; }}&lt;/span&gt;&lt;/pre&gt;
			&lt;/td&gt;
		&lt;/tr&gt;
			&lt;/tbody&gt;
&lt;/table&gt;
&lt;/div&gt;
&lt;p/&gt;
&lt;p&gt;&lt;tt&gt;parts-&amp;gt;assembled.command&lt;/tt&gt; is the final assembled command to send. This either points to &lt;tt&gt;parts-&amp;gt;body&lt;/tt&gt; if no options are required to be appended to the base command, or is set to &lt;tt&gt;parts-&amp;gt;assembled_body&lt;/tt&gt; otherwise. (I think perhaps this is a micro optimization, to avoid copying the command if we don&apos;t need to.)&lt;/p&gt;

&lt;p&gt;So, at the start of the &lt;tt&gt;mongoc_cmd_parts_assemble&lt;/tt&gt; there is:&lt;/p&gt;
&lt;p/&gt;
&lt;div id=&quot;syntaxplugin&quot; class=&quot;syntaxplugin&quot; style=&quot;border: 1px dashed #bbb; border-radius: 5px !important; overflow: auto; max-height: 30em;&quot;&gt;
&lt;table cellspacing=&quot;0&quot; cellpadding=&quot;0&quot; border=&quot;0&quot; width=&quot;100%&quot; style=&quot;font-size: 1em; line-height: 1.4em !important; font-weight: normal; font-style: normal; color: black;&quot;&gt;
		&lt;tbody &gt;
				&lt;tr id=&quot;syntaxplugin_code_and_gutter&quot;&gt;
						&lt;td  style=&quot; line-height: 1.4em !important; padding: 0em; vertical-align: top;&quot;&gt;
					&lt;pre style=&quot;font-size: 1em; margin: 0 10px;  margin-top: 10px;   margin-bottom: 10px;  width: auto; padding: 0;&quot;&gt;&lt;span style=&quot;color: black; font-family: &apos;Consolas&apos;, &apos;Bitstream Vera Sans Mono&apos;, &apos;Courier New&apos;, Courier, monospace !important;&quot;&gt;parts-&amp;gt;assembled.command = parts-&amp;gt;body;&lt;/span&gt;&lt;/pre&gt;
			&lt;/td&gt;
		&lt;/tr&gt;
			&lt;/tbody&gt;
&lt;/table&gt;
&lt;/div&gt;
&lt;p/&gt;

&lt;p&gt;And whenever an option is set, you&apos;re supposed to call &lt;br/&gt;
&lt;tt&gt;_mongoc_cmd_parts_ensure_copied&lt;/tt&gt;, which switches it over:&lt;/p&gt;
&lt;p/&gt;
&lt;div id=&quot;syntaxplugin&quot; class=&quot;syntaxplugin&quot; style=&quot;border: 1px dashed #bbb; border-radius: 5px !important; overflow: auto; max-height: 30em;&quot;&gt;
&lt;table cellspacing=&quot;0&quot; cellpadding=&quot;0&quot; border=&quot;0&quot; width=&quot;100%&quot; style=&quot;font-size: 1em; line-height: 1.4em !important; font-weight: normal; font-style: normal; color: black;&quot;&gt;
		&lt;tbody &gt;
				&lt;tr id=&quot;syntaxplugin_code_and_gutter&quot;&gt;
						&lt;td  style=&quot; line-height: 1.4em !important; padding: 0em; vertical-align: top;&quot;&gt;
					&lt;pre style=&quot;font-size: 1em; margin: 0 10px;  margin-top: 10px;   margin-bottom: 10px;  width: auto; padding: 0;&quot;&gt;&lt;span style=&quot;color: black; font-family: &apos;Consolas&apos;, &apos;Bitstream Vera Sans Mono&apos;, &apos;Courier New&apos;, Courier, monospace !important;&quot;&gt;parts-&amp;gt;assembled.command = &amp;amp;parts-&amp;gt;assembled_body;&lt;/span&gt;&lt;/pre&gt;
			&lt;/td&gt;
		&lt;/tr&gt;
			&lt;/tbody&gt;
&lt;/table&gt;
&lt;/div&gt;
&lt;p/&gt;

&lt;p&gt;My guess is that this is a micro-optimization, to avoid copying parts-&amp;gt;body if no options were set and nothing changed. However, we don&apos;t always call &lt;tt&gt;_mongoc_cmd_parts_ensure_copied&lt;/tt&gt; every time options are set, meaning we might omit sending them. I don&apos;t think it&apos;s currently possible for this to surface because this only applies when:&lt;/p&gt;
&lt;ul class=&quot;alternate&quot; type=&quot;square&quot;&gt;
	&lt;li&gt;no $lsid (session ID) is appended&lt;/li&gt;
	&lt;li&gt;we&apos;re using OP_MSG (so, post 3.6)&lt;/li&gt;
	&lt;li&gt;user manually specifies $db in the command document (which is probably pretty rare)&lt;/li&gt;
&lt;/ul&gt;


&lt;p&gt;Because of &lt;a href=&quot;https://jira.mongodb.org/browse/CDRIVER-3070&quot; title=&quot;Do not include lsid in commands if topology does not support sessions&quot; class=&quot;issue-link&quot; data-issue-key=&quot;CDRIVER-3070&quot;&gt;&lt;del&gt;CDRIVER-3070&lt;/del&gt;&lt;/a&gt;, we always (incorrectly) assume a 3.6+ server supports sessions and append the $lsid. When that&apos;s fixed, we&apos;ll need to fix this.&lt;/p&gt;</description>
                <environment></environment>
        <key id="890513">CDRIVER-3303</key>
            <summary>Some options not applied for OP_MSG without sessions</summary>
                <type id="3" iconUrl="https://jira.mongodb.org/secure/viewavatar?size=xsmall&amp;avatarId=14718&amp;avatarType=issuetype">Task</type>
                                            <priority id="3" iconUrl="https://jira.mongodb.org/images/icons/priorities/major.svg">Major - P3</priority>
                        <status id="6" iconUrl="https://jira.mongodb.org/images/icons/statuses/closed.png" description="The issue is considered finished, the resolution is correct. Issues which are closed can be reopened.">Closed</status>
                    <statusCategory id="3" key="done" colorName="success"/>
                                    <resolution id="13201">Fixed</resolution>
                                        <assignee username="roberto.sanchez@mongodb.com">Roberto Sanchez</assignee>
                                    <reporter username="kevin.albertson@mongodb.com">Kevin Albertson</reporter>
                        <labels>
                    </labels>
                <created>Sat, 10 Aug 2019 20:09:51 +0000</created>
                <updated>Sat, 28 Oct 2023 11:29:10 +0000</updated>
                            <resolved>Thu, 7 May 2020 01:35:32 +0000</resolved>
                                                    <fixVersion>1.17.0-beta2</fixVersion>
                    <fixVersion>1.17.0</fixVersion>
                                    <component>libmongoc</component>
                                        <votes>0</votes>
                                    <watches>1</watches>
                                                                                                                <comments>
                            <comment id="3106178" author="xgen-internal-githook" created="Mon, 25 May 2020 21:14:18 +0000"  >&lt;p&gt;Author:&lt;/p&gt;
{&apos;name&apos;: &apos;Roberto C. S&#225;nchez&apos;, &apos;email&apos;: &apos;roberto@connexer.com&apos;, &apos;username&apos;: &apos;rcsanchez97&apos;}
&lt;p&gt;Message: &lt;a href=&quot;https://jira.mongodb.org/browse/CDRIVER-3303&quot; title=&quot;Some options not applied for OP_MSG without sessions&quot; class=&quot;issue-link&quot; data-issue-key=&quot;CDRIVER-3303&quot;&gt;&lt;del&gt;CDRIVER-3303&lt;/del&gt;&lt;/a&gt; always apply command options for OP_MSG&lt;br/&gt;
Branch: r1.17&lt;br/&gt;
&lt;a href=&quot;https://github.com/mongodb/mongo-c-driver/commit/bbd2669545e582934d5f29512901bc066cd3f546&quot; class=&quot;external-link&quot; target=&quot;_blank&quot; rel=&quot;nofollow noopener&quot;&gt;https://github.com/mongodb/mongo-c-driver/commit/bbd2669545e582934d5f29512901bc066cd3f546&lt;/a&gt;&lt;/p&gt;</comment>
                            <comment id="3073059" author="xgen-internal-githook" created="Thu, 7 May 2020 01:31:46 +0000"  >&lt;p&gt;Author:&lt;/p&gt;
{&apos;name&apos;: &apos;Roberto C. S&#225;nchez&apos;, &apos;email&apos;: &apos;roberto@connexer.com&apos;, &apos;username&apos;: &apos;rcsanchez97&apos;}
&lt;p&gt;Message: &lt;a href=&quot;https://jira.mongodb.org/browse/CDRIVER-3303&quot; title=&quot;Some options not applied for OP_MSG without sessions&quot; class=&quot;issue-link&quot; data-issue-key=&quot;CDRIVER-3303&quot;&gt;&lt;del&gt;CDRIVER-3303&lt;/del&gt;&lt;/a&gt; always apply command options for OP_MSG&lt;br/&gt;
Branch: master&lt;br/&gt;
&lt;a href=&quot;https://github.com/mongodb/mongo-c-driver/commit/a4f02b73833b2421098e55dc2ad159ccd97cef5d&quot; class=&quot;external-link&quot; target=&quot;_blank&quot; rel=&quot;nofollow noopener&quot;&gt;https://github.com/mongodb/mongo-c-driver/commit/a4f02b73833b2421098e55dc2ad159ccd97cef5d&lt;/a&gt;&lt;/p&gt;</comment>
                            <comment id="2365803" author="kevin.albertson" created="Sat, 10 Aug 2019 20:11:16 +0000"  >&lt;p&gt;Here is a branch with a repro (with a workaround for &lt;a href=&quot;https://jira.mongodb.org/browse/CDRIVER-3070&quot; title=&quot;Do not include lsid in commands if topology does not support sessions&quot; class=&quot;issue-link&quot; data-issue-key=&quot;CDRIVER-3070&quot;&gt;&lt;del&gt;CDRIVER-3070&lt;/del&gt;&lt;/a&gt;)&lt;br/&gt;
&lt;a href=&quot;https://github.com/kevinAlbs/mongo-c-driver/tree/cdriver3303&quot; class=&quot;external-link&quot; target=&quot;_blank&quot; rel=&quot;nofollow noopener&quot;&gt;https://github.com/kevinAlbs/mongo-c-driver/tree/cdriver3303&lt;/a&gt;&lt;/p&gt;</comment>
                    </comments>
                <issuelinks>
                            <issuelinktype id="10011">
                    <name>Depends</name>
                                                                <inwardlinks description="is depended on by">
                                        <issuelink>
            <issuekey id="729154">CDRIVER-3070</issuekey>
        </issuelink>
                            </inwardlinks>
                                    </issuelinktype>
                            <issuelinktype id="10012">
                    <name>Related</name>
                                            <outwardlinks description="related to">
                                        <issuelink>
            <issuekey id="1342729">CDRIVER-3657</issuekey>
        </issuelink>
                            </outwardlinks>
                                                        </issuelinktype>
                    </issuelinks>
                <attachments>
                    </attachments>
                <subtasks>
                    </subtasks>
                <customfields>
                                                                                                                                                                                                                                                                                                                                                                    <customfield id="customfield_15850" key="com.atlassian.jira.plugins.jira-development-integration-plugin:devsummary">
                        <customfieldname>Development</customfieldname>
                        <customfieldvalues>
                            
                        </customfieldvalues>
                    </customfield>
                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                            <customfield id="customfield_12550" key="com.pyxis.greenhopper.jira:gh-lexo-rank">
                        <customfieldname>Rank</customfieldname>
                        <customfieldvalues>
                            <customfieldvalue>2|hv8xs7:</customfieldvalue>

                        </customfieldvalues>
                    </customfield>
                                                                <customfield id="customfield_10558" key="com.pyxis.greenhopper.jira:gh-global-rank">
                        <customfieldname>Rank (Obsolete)</customfieldname>
                        <customfieldvalues>
                            <customfieldvalue>9223372036854775807</customfieldvalue>
                        </customfieldvalues>
                    </customfield>
                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                            </customfields>
    </item>
</channel>
</rss>