Repository navigation
Update dependency nokogiri to v1.19.5 - #3658
Merged
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This PR contains the following updates:
1.19.4→1.19.5Release Notes
sparklemotion/nokogiri (nokogiri)
v1.19.5Compare Source
Security
Node#[]=orNode#lang=while Ruby objects still reference the attribute's children. See GHSA-3w22-6jfg-527w for more information.noscriptunless the document was parsed withparse_noscript_content_as_text: true, preventing a mutation XSS in applications that allownoscriptin sanitized HTML. A fragment now parsesnoscriptcontent the same way as its document, and raisesArgumentErrorif given a conflictingparse_noscript_content_as_text:value. Comments and raw-text elements insidenoscriptare still serialized unchanged, as the HTML standard requires, so applications that sanitize HTML and allownoscriptshould parse withparse_noscript_content_as_text: trueor removenoscriptelements. See GHSA-4p39-36wj-23h4 for more information.NONETnow blocks external DTDs and entities whose systemId uses a short-form IPv4 host such as127.1, or is an absolute path in a document parsed from a network URL. Previously Nokogiri fetched them over the network during a parse withDTDLOADorNOENT, potentially enabling SSRF. See GHSA-v995-44v6-5fqm for more information.Improved
ALLOCV_Ninstead ofrb_gc_register_address. (#3648) @jhawthornFixed
Node#namespace=now declares the namespace on the node when no declaration of its prefix is in scope there. Previously, on CRuby, a node given a namespace and then moved under an element that binds the same prefix, or the default namespace, to a different URI was serialized in that other namespace. On JRuby, the namespace was missing fromNode#namespace_definitions. (#3459, #3462) @flavorjonesXML::Readeris no longer affected by changes made to its source String after the reader is created. Previously, such a change could make the reader parse the changed String on JRuby, or changed or freed memory on CRuby with libxml2 < 2.11. (#3694) @flavorjonesNode#namespace = falsenow raisesTypeError. Previously it crashed the process on CRuby and raisedClassCastExceptionon JRuby. (#3719) @flavorjonesXML::XPathContext.newandXML::XPathContext#node=raiseTypeErrorwhen the argument is not aNode. Previously, on CRuby, most such arguments crashed the process. (#3720) @flavorjonesan+bselectors with a zero or negative step now follow the Selectors spec::nth-child(0n+3)matches only the third element (previously nothing),0n-1matches nothing instead of raisingZeroDivisionError, and-n+0/-2nmatch nothing instead of every (or every other) element. @afurm#id,.classand attribute selectors whose value contains a quote character, such as#it\'sor[a=\'b\'], now match that value. Previously they raisedNokogiri::XML::XPath::SyntaxErroror generated an XPath query with a different meaning. An#idselector containing an escaped newline no longer drops the text after it. (#3724, #3727) @flavorjones:contains()arguments, custom pseudo-class string arguments, and element names containing a CSS escape, such asp:contains('it\'s')ora\-b, now match. Previously they raisedNokogiri::XML::XPath::SyntaxError. An element name that is not an XML name after unescaping, such asa\!b, raisesNokogiri::CSS::SyntaxError, except as an unprefixed name in an HTML5 document, where it matches by local name. In ids, classes, attribute values, element names and string arguments, an escape for U+0000, a surrogate, or a code point above U+10FFFF now decodes to U+FFFD, and a vertical tab after a hexadecimal escape is no longer dropped, as CSS Syntax Level 3 requires. (#3728, #3729) @flavorjones[a\-b], now match. Previously they raisedNokogiri::XML::XPath::SyntaxError. An unprefixed attribute name that cannot be written as an XPath name test after unescaping, such as[\@click]or[\:class], matches an attribute with that exact name in no namespace. The HTML parsers create such attributes. A prefixed one, such as[x|a\!b], raisesNokogiri::CSS::SyntaxError. (#3732, #3735) @flavorjones<and>in attribute values as<and>, following whatwg/html#6362. Previously they were written literally. (#3529, #3741) @flavorjonesReader.outer_xmland.inner_xmlproperly capture syntax errors. (#3558) @flavorjonesXML::SAX::PushParserandHTML4::SAX::PushParserkeep parsing correctly after GC compaction relocates their SAX parser. (#3665) @zankerXML::Readerkeeps reading correctly when GC compaction runs. Previously, compaction could move the IO or String the reader was created from, and the reader then crashed or parsed the wrong bytes. (#3666, #3668) @zankerXSLT::Stylesheet#transformcalls custom XSLT extension functions correctly when GC compaction runs. Previously, after compaction moved a stylesheet that uses an extension module, a later transform raised an exception or crashed. (#3667) @jeremyXSLT::Stylesheet#transformpasses the correct params when GC compaction runs while it converts a param with#to_str. Previously, compaction could move the bytes of earlier short params, and the transform raisedRuntimeErroror silently left a param empty. (#3670) @flavorjonesXML::SAX::ParserContext.ioandHTML4::SAX::ParserContext.iokeep parsing correctly after GC compaction relocates the IO. (#3669) @zankerDocument#canonicalizepasses the correct inclusive namespace prefixes when it converts a prefix with#to_str, and when GC compaction runs during canonicalization. Previously, libxml2 could read freed or moved memory, and the output lost namespaces or Ruby crashed. Raising on a prefix with a null byte also no longer leaks memory. (#3714, #3750) @Priyank-gouda678 @flavorjonesXSLT::Stylesheet#transformcalls custom XSLT extension functions correctly when GC compaction runs during the transform. Previously, compaction could move an extension handler instance, and a later call to one of its functions raised an exception or crashed. (#3748) @flavorjonesXSLT::Stylesheet#transformcalls custom XSLT extension functions correctly after an extension function runs another transform with the same stylesheet, or while another thread does. Previously, the inner or concurrent transform released extension handler instances that the other transform still used, and GC could free them. (#3748) @flavorjonesXML::Reader.from_memoryaccepts an object that responds toto_str. Previously, it could raiseNokogiri::XML::SyntaxError. (#3694) @flavorjonesXPathContext#evaluateno longer leaks memory when a custom XPath function handler raises an exception. (#3715, #1610) @flavorjonesXPathContext#evaluatenow resets libxml2's error handlers and unregisters the custom function lookup. Previously, libxml2 could keep references to garbage-collected Ruby objects. (#3715) @flavorjonesgumbo_parse_with_optionsno longer crashes whenstop_on_first_erroris set and the parse ends on an unknown end tag in a fragment context. This is a C API path; it is not reachable from Ruby, because Nokogiri never setsstop_on_first_error. (#3672) @jeremyNode#namespace_definitionsso that it now behaves identically to CRuby. (#2543, #3460) @flavorjonesTypeErrorwhen the context node or one of its ancestors is an element whosenamereturns something other than aString. Previously this could crash the process. (#3740) @flavorjones</body>and a<frameset>start tag follows the comment. (#3744) @flavorjonesreadraises an exception that is not aStandardError, such asInterruptorTimeout::ExitException, parsing withXML::Document,HTML4::DocumentorXML::Readerno longer leaks memory or leaves behind a stale libxml2 error handler. (#3718, #3721) @Priyank-gouda678 @flavorjonesrb_len_ttype. Previously compilation failed with an incompatible pointer type error inxml_node_set.c. (#3711) @larskanisNode#line=now raisesNoMethodErroronDocument,DTD,Attr,ElementDecl,AttributeDeclandEntityDecl, as#linealready does on all of them exceptAttr. Previously it wrote past the end of the node's memory or overwrote an unrelated field, which could crash the process. (#3737, #3738) @Priyank-gouda678 @flavorjonesDocument#create_elementandNode.newno longer set the namespace to the document's default namespace. The namespace must be set explicitly withnamespace=or by parenting the node. (#3457, #3463) @flavorjonesWEB-INF/lib. Previously they raisedDTMConfigurationException: No default implementation found. Nokogiri also no longer sets the JVM-wide system propertyorg.apache.xml.dtm.DTMManager, which made XSLT transformations and other Xalan users in the same JVM fail in that setup. (#3707) @flavorjonesConfiguration
📅 Schedule: (UTC)
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR is behind base branch, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.
This PR was generated by Mend Renovate. View the repository job log.