Cloudflare Deploys Really Slow Code, Takes Down Entire Company

Поділитися
Вставка
  • Опубліковано 21 лис 2024

КОМЕНТАРІ • 785

  • @rockyvillano777
    @rockyvillano777 Рік тому +1806

    If I had a nickel for every time quadratic complexity passed testing but blew up in prod I'd be rich

    • @allyourpie4323
      @allyourpie4323 Рік тому +196

      What if every time it quadratic complexity passed testing, you got a nickel for every time quadratic complexity passed testing?

    • @renakunisaki
      @renakunisaki Рік тому +120

      If I had a nickel for every time I got a nickel...

    • @Bollibompa
      @Bollibompa Рік тому +47

      If I got a nickel for every time someone says "If I got a nickel" I would be a gabbagillionaire!

    • @Aaron.Thomas
      @Aaron.Thomas Рік тому +22

      This is why we have and should have computer science and not just computer engineering - math is important.

    • @yaksher
      @yaksher Рік тому +19

      This is exponential, not quadratic. If it were quadric, there wouldn't be any problems.

  • @eldrago19
    @eldrago19 Рік тому +150

    "Some programmers run into a problem and think, 'I will use regex to solve this!' Now they have two problems."
    - Zawinski

  • @xorinzor
    @xorinzor Рік тому +2474

    Despite all this, I still very much love Cloudflare especially because of their transparency. They always go into great depth explaining what happened, what they did, and how they resolved it.
    Many companies can learn a thing or 2 from them in that regard. Customers tend to have more faith in a company that just owns up to it's mistakes rather then trying to have a PR department cover it up in nice words.

    • @StkyDkNMeBlz
      @StkyDkNMeBlz Рік тому +93

      ​@@KabodankiCaptcha? Google the "Privacy Pass" extension. It lets you skip the tests by doing tests beforehand.

    • @jacksoncremean1664
      @jacksoncremean1664 Рік тому +37

      @@StkyDkNMeBlz they don't have captcha anymore, they use turnstile

    • @maskettaman1488
      @maskettaman1488 Рік тому +121

      I love my state sponsored man in the middle

    • @returndislikes6906
      @returndislikes6906 Рік тому

      I hate cloudflare because its trying to become monopoly on internet ethics. Its not your job to pass judgement what is allowed on the internet. Banning something that is illegal is fair. But banning because its immoral according to them... yea I hate cloudflare.

    • @returndislikes6906
      @returndislikes6906 Рік тому +51

      @@StkyDkNMeBlz I really want to be tracked by Google across the internet with their corpo issued cryptographic IDs. You do understand what you are shilling?

  • @nwrocketman6438
    @nwrocketman6438 Рік тому +1812

    2:51 I like how the 1000x engineer just foreshadows all the events that about to happen, and then approves the change.

    • @tehroflcopters
      @tehroflcopters Рік тому +120

      par for the course tbh

    • @1008OH
      @1008OH Рік тому +309

      Yeah average senior software engineer moment tbh

    • @jfbeam
      @jfbeam Рік тому +141

      @EgonFreeman From experience... had they spoken up, they would almost certainly have been met with one of "who asked you?", "shut up, we're doing this!", or just ignored. Dozens of others didn't notice any issues, so who's going to listen to "that guy"?

    • @xhivo97
      @xhivo97 Рік тому +29

      a true 1000x enginerd would not use regex lol

    • @MiesvanderLippe
      @MiesvanderLippe Рік тому +70

      @@jfbeam sounds like the culture in your organisation is arse. A let’s be careful should be taken serious coming from a day 1 intern.

  • @EndMaster0
    @EndMaster0 Рік тому +82

    I guarantee all the engineers who reviewed it didn't even look at the regex. you don't go poking someone elses regex

    • @TheLordoftheDarkness
      @TheLordoftheDarkness 5 місяців тому +25

      Or even your own if you wrote it more than 1 week ago.

    • @_torgeek9108
      @_torgeek9108 3 місяці тому +1

      Lol I hate that I’m relating to this😂🤣😂

  • @evilsqirrel
    @evilsqirrel Рік тому +159

    I work in a cybersecurity administration space where regex is used all the time as a necessity. This is a story we tell people all the time to make sure they understand how important it is to make efficient regex.

    • @Wyvernnnn
      @Wyvernnnn Рік тому +1

      The regex is fine there was no reason for the engine to backtrack on it

    • @Zei33
      @Zei33 Рік тому +15

      @@WyvernnnnI disagree. The pattern didn’t make much sense. It was clearly missing something between the initial wildcard and non-capturing group. There’s never any reason to put two wildcards next to each other like that.

    • @Wyvernnnn
      @Wyvernnnn Рік тому +6

      @@Zei33 Yeah that was weird, but it should still get O(n) in the end, that's the whole point of regular expressions (as long as you don't have capturing groups that can be re-used within the regex aka backtracking)

    • @Croz89
      @Croz89 Рік тому +4

      I can understand how it happened, even an experienced programmer can struggle to parse a regex by eye and it's easy to make something that's a resource hog without realising. It's certainly a lesson to test your regex thoroughly before release.

    • @Zei33
      @Zei33 Рік тому

      @@Croz89 oh yeah I’m constantly making mistakes. No one is perfect, that’s why debugging and beta testing exists. After over a decade of programming, I don’t make a lot of mistakes, but when I do they’re usually obscure cases or subtle logic errors. When you’re working with tens of thousands of lines of code and looking at them for 8 to 20 hours a day, mistakes are gonna happen.

  • @ejcx_
    @ejcx_ Рік тому +624

    I was actually at Cloudflare in the room for Cloudbleed and this issue, in SF for Cloudbleed and happened to be in London for this one. The real story is much better than this. We were at lunch doing a tech talk in the lunchroom when someone grabbed the mic and announced we were having a P0. We stampeded back to our desks and got to work fixing it. The issue was obviously related to the WAF from the start and it was just a matter of cleaning up. Keep up the videos they are great

    • @Nayayom
      @Nayayom Рік тому +12

      So how's working with cloudflare like ? :)

    • @0xggbrnr
      @0xggbrnr Рік тому +107

      @@NayayomIt’s actually really awesome. Tons of really smart, kind, curious people. Everything internally is about transparency, execution, and learning. Definitely engineering-centric, but also super product-focused in that the customer is always considered during meetings/talks/decisions.

    • @Nayayom
      @Nayayom Рік тому +23

      @@0xggbrnr sounds like a good place to work! Glad to hear that

    • @CarlosISoares
      @CarlosISoares Рік тому +11

      Ok, but is not BETTER than the video story lol

    • @magno5157
      @magno5157 Рік тому +10

      What happened to the employee who made the regex?

  • @CYXXYC
    @CYXXYC Рік тому +670

    2:37 laughed my ass off at "delete master after the pull request is merged"

    • @ABaumstumpf
      @ABaumstumpf Рік тому +41

      and there are people that would click that checkbox.

    • @Blast-Forward
      @Blast-Forward Рік тому +61

      Chill, "master" is the dev branch here while "main" is the master branch of course.

    • @zerto111
      @zerto111 Рік тому +29

      @@Blast-Forward Let's just agree that it's a funny easter egg in the video and laughing i justified.

    • @tisaconundrum
      @tisaconundrum Рік тому +4

      Glad to see others had seen that too 😂❤

    • @MrWarlock616
      @MrWarlock616 Рік тому

      Yeah that was such a nice touch!😂

  • @mwissel
    @mwissel Рік тому +196

    Love the little details, like the upside down cloudflare icon in Australia. Good job editing!

  • @greg_289
    @greg_289 Рік тому +428

    Worth mentioning that Cloudflare isn’t just a CDN. It’s predominantly used by most websites as a web proxy responsible for the majority if not all requests to the origin.

    • @emeraldbonsai
      @emeraldbonsai Рік тому +22

      the webproxy is a cdn last i checked

    • @VeggieRice
      @VeggieRice Рік тому +3

      users can elect their own dns service

    • @hoo2042
      @hoo2042 Рік тому +20

      @@emeraldbonsai Typically a CDN serves static or at least mostly static data. A CDN may be implemented as a caching web proxy, but a web proxy can do a lot more than what usually falls under the definition of "CDN". In CloudFlare's case, they basically offer both and blur the line about which is which, which is fine since it's a blurry line, but the person you are replying to isn't wrong.

    • @hoo2042
      @hoo2042 Рік тому +6

      @@VeggieRice ⁠DNS has nothing to do with what's being discussed here (aside from being an earlier step in the chain that would take you to the page's configured web proxy or CDN, of course, but equivalently so to saying "the user can elect their own browser").

    • @benhook1013
      @benhook1013 Рік тому +8

      The distinction is useful here as the outage is much more impactful if your web pages wont even load themselves (because the web proxy is down) rather than just CDN assets not loading (which could be only large assets).

  • @str4ng3-b34ut1ful
    @str4ng3-b34ut1ful Рік тому +233

    It's important to note that re2 actually has other downsides compared to other regex engines, such as being unable to handle lookaheads and lookbehinds. This isn't just an implementation issue either: adding these operations actually makes regex strictly stronger than a finite state machine (instead it becomes a pushdown automaton). There's also a lot of fun math with finite state machines, where it turns out they're strictly equivalent to generating functions, which are basically power series where you don't care about convergence!

    • @Ceelvain
      @Ceelvain Рік тому +6

      I think "look-around" assertions could still be implemented to run in linear time. As far as I know, back references is the only feature that can make the run time go exponential. In fact, matching regexes with backrefs is proven NP-hard.

    • @hoo2042
      @hoo2042 Рік тому +21

      Lacking some of the more advanced PCRE features in order to make guarantees about the maximum runtime seems like the right compromise to make for a high-volume security frontend that sits between the global population and a large swath of the internet.

    • @Ceelvain
      @Ceelvain Рік тому +23

      @@hoo2042 That's actually why Russ Cox developed RE2 in the first place. He made it for google code search (now defunct). You can't really be Google and expect tech people to only input well-behaved regexes. He has a very interesting series of articles named "Implementing Regular Expressions". I really recommend every developer to read them.

    • @tommihommi1
      @tommihommi1 Рік тому +2

      shouldn't regex expanded like this be called cfex instead, since it's, well, no longer a regular expression

    • @mustard96
      @mustard96 Рік тому +5

      @@hoo2042The problem now is that they use it on every Google product. re2 is the regex engine of BigQuery and I’m stuck with this limitations. It doesn’t make sense in a data warehouse.

  • @Thect
    @Thect Рік тому +235

    "like re2, which work by converting regex to a state machine, or fancy computer science flowcharts"
    Damn, I wish I can say this line to the professor who teaches compiler course in my university lol

    • @mr_confuse
      @mr_confuse Рік тому +20

      I had straight vietnam flashbacks when the statemachine came up lmao

    • @skyhappy
      @skyhappy Рік тому +6

      State machines were the most useless thing I learned 2nd year in "computational theory" class. Whole class was academic fluff.

    • @tissuepaper9962
      @tissuepaper9962 Рік тому +58

      ​@@skyhappy FSMs are used everywhere, they're the basic building block of most digital protocols and embedded systems. Definitely not "useless".

    • @DNX3M
      @DNX3M Рік тому +3

      I call them spaghetti meatballs. Feel free to use that one.

    • @arthurpenndragon6434
      @arthurpenndragon6434 Рік тому +32

      @@skyhappy average framework enthusiast with no understanding of computer science.

  • @Kabodanki
    @Kabodanki Рік тому +16

    upside down cloudflare logo for australia was gold

  • @andrewallbright
    @andrewallbright Рік тому +72

    These stories are so cathartic. Thanks for applying your storytelling to these niche topics!

  • @thewhitefalcon8539
    @thewhitefalcon8539 Рік тому +44

    Someone wrote a paper ages ago about backtracking vs non-backtracking regex engines and the state of software slowness...
    The title is "Regular Expression Matching Can Be Simple And Fast (but is slow in Java, Perl, PHP, Python, Ruby, ...)" written by a Russ Cox in 2007. I bet he's feeling vindicated

    • @lhpl
      @lhpl Рік тому +7

      That article, and several others, should be mandatory reading for anyone using regular expressions.

  • @Aldrasio
    @Aldrasio Рік тому +43

    This is why as a general rule I NEVER use .* in my regexes. If I want to match everything before an equals sign, I'd use [^=]*= rather than .*= because it's always better to be as explicit as possible.

    • @framegrace1
      @framegrace1 Рік тому +2

      But that would match just the first '=' , not all of them. If you have a lot of parameters on a URL , you will have a lot of '=' and you will want to search all of them for certain things.

    • @HenryLoenwind
      @HenryLoenwind Рік тому +11

      ​@@framegrace1 That's why you don't anchor the expression to the end of the string in this case. We don't care what else is at the end of the URL if we find a "bad thing" near the start. Also, most regex engines have a shortcut implementation for regexes ending in ".*"/".*$", so the one at the end is of no concern.
      And BTW, the issue was mostly the ".*.*", not so much the ".*=". Backtracking the latter isn't so expensive---it doesn't really matter if the engine has to search for the = from the start of the end of the remaining string. It most likely has a shortcut for "fixed character after match all" anyway. There's a good chance that ".*?=" is faster than "[^=]*?="/"[^=]*=" as it can scan the string using a simple "equals" comparison and be done. This, however, all goes out the window once there are multiple ways to match, like the infamous ".*.*". So when using this optimisation on purpose, it makes sense to manually commit after the "=" (e.g. with "(*COMMIT)").

    • @AbiGail-ok7fc
      @AbiGail-ok7fc Рік тому +3

      @@framegrace1 You can still get the last '=' by being more explicit: /([^=]*=)*=/ or /=[^=]*$/

  • @MindLaboratory
    @MindLaboratory Рік тому +185

    It's amazing how casually things are actually handled behind the scenes in the IT world. I once wrote some software for a bank, did a 3 hour audit of the code with 5 of their top developers, after which they installed a pre-compiled earlier test version on their prod system. smh

    • @jnz007
      @jnz007 Рік тому +1

      😂

    • @DanielSmedegaardBuus
      @DanielSmedegaardBuus Рік тому +66

      I once fixed a globally crashing iOS app by hacking the backend to send out technically incorrect data. The app passed all tests because the test suites didn't include any data to reveal division-by-zero bugs.
      This was especially bad since the time to get Apple to review and deploy an updated version could take a week or more, IIRC.
      After conversing with the dev responsible, I asked him how he habdled fractional numbers, and he was sure that fractional numbers were always displayed as integers, so I changed the API to send instances of 0 as 0.001, effectively circumventing the bug while displaying calculated numbers (and 0s) correctly in the app.
      I think it's the most hacky fix I've ever deployed. It felt terrible and exhilarating and awesome all at the same time 😂 I'm actually a little proud 😇

    • @SSM24_
      @SSM24_ 3 місяці тому +1

      @@DanielSmedegaardBuus That's awful, I love it.

  • @royalepros669
    @royalepros669 Рік тому +86

    my brain automatically shut down when you start explaining the regex...

    • @TMRick1
      @TMRick1 Рік тому +2

      Same here dude. I can't understand how people are still relying on regex for such important aspects of the code. It's just mind-blowing that a firewall rule is managed with that in 2023.

    • @arielcg_
      @arielcg_ Рік тому +21

      @@TMRick1 what alternative is there that is universally supported and has the same level of flexibility for how "compact" it is?

    • @tissuepaper9962
      @tissuepaper9962 Рік тому +22

      ​@@TMRick1 you just don't know the pain of using anything else to do what regex can do. What do you suggest? awk?

    • @Игор-ь9щ
      @Игор-ь9щ Рік тому +3

      lol just write your own parser. You are acting as if that's a hard problem to solve and as if customers are not important. You just want to make your lives as "programmers" easier. Have some responsibility for the unnecessary amount of code that runs on users machines.

    • @jtfoog5220
      @jtfoog5220 9 місяців тому

      @@Игор-ь9щI use arch btw lol

  • @UnrealOG137
    @UnrealOG137 Рік тому +19

    3:50 Putting cloudlfare upside down above Australia was a hilarious touch!

  • @spicybaguette7706
    @spicybaguette7706 Рік тому +15

    They cannot be DDOSed, but they can DDOS themselves...

  • @pav431
    @pav431 Рік тому +52

    I work as a sysadmin, yet I wish I had this much insight into how all the technologies I use daily work this deeply. I never finished my college, so I only ever heard of DFA, but despite that, your video explained it very well, and showed how much of an issue a simple regex can be, when executed thousands a times a second.
    Please make more videos, I cannot wait for more

    • @Atlantis357
      @Atlantis357 Рік тому +9

      It is interesting to note that any regex can be represented as a nondeterministic finite automata (NFA) and any NFA can be converted into a DFA using a simple algorithm. The only downside is that the DFA may end up with exponentially more states than the NFA which can take up a lot of memory.

    • @Nayayom
      @Nayayom Рік тому +4

      Good thing you enjoyed it, i have not so fun memories doing DFAs and NFAs by hand on college. :(

    • @habama1077
      @habama1077 Рік тому +1

      ​@@Nayayombro same. I hated it. But understanding them is pretty good. We can see how google's devs used
      automata and formal grammar theory to develop a useful practical application with regex.

    • @lhpl
      @lhpl Рік тому +4

      @pav431 even without some formal education, I'd say anyone working as sysadmin should know something about algorithms and complexity theory. Especially when writing code that systems used by others depend on. And shell scripts _are_ code.
      Knowing that there are regular expressions, and highly irregular expressions that are just _called_ something like "Perl compatible regular expressions" or "extended regexp" or whatever, is important. So is not writing script that unnecessarily nest three or more loops, and work fine on small testdata, but take "forever" with realistic sizes of data. Know and understand the various o-notations. Just because quicksort is usually quick, doesn't prevent it from having O(n²) worst case complexity. You may be fine with that, but you will want to know why you can live with it. There has to be a metric ton of good books on this, so it's possible to learn. Enjoy!

    • @Aaron.Thomas
      @Aaron.Thomas Рік тому

      ​@@lhpl As someone who's inherited, and then had to completely rewrite from scratch, core automation scripts for clusters that were written by novice sysadmins, I concur that learning these things is important.
      Some aysadmins learn that awk and grep exist and that's the end of their training. Rewriting from scratch saved me hours upon hours it would have cost me to try to maintain the poorly made code inherited from novice sysadmins.

  • @MrMCMaxLP
    @MrMCMaxLP Рік тому +7

    Ken Thompson is crying really hard. His work has been around for decades. As a CS guy who has specialized in algorithms, this hurts in the middle of the heart.

  • @LordMegatherium
    @LordMegatherium Рік тому +44

    Regex is great like shell scripts: works everywhere and does it jobs... up until a certain script size when the chance of bugs starts increasing and you should think of using another tool instead or in conjunction.
    Also this sounds like GitOps to the extreme: when you can only change your state via your repo and all the triggers that come with it you might as well replace your CD with a single bash script (see above).

  • @chewcodes
    @chewcodes Рік тому +151

    this is my new favorite channel. explaining everything clearly, and being humorous with small jokes here and there

    • @gblargg
      @gblargg Рік тому +20

      And explosions, lots of explosions.

    • @princesssshortie
      @princesssshortie Рік тому +10

      Same. And the upside-down cloudflare logo on Australia killed me. 😂

  • @glitchy_weasel
    @glitchy_weasel Рік тому +10

    Absolutely crazy videos you're pumping up. Love your comedic editing style too!
    Every video of yours makes me feel like to the entire internet could break at any moment lol

  • @RandomFish-gx7pj
    @RandomFish-gx7pj Рік тому +5

    It's kinda funny that the Internet was designed to be a `web` that hopefully would prevent failures of a single node taking down the whole system, but nowadays we heavily rely on a handful of service providers just to run the Internet.

  • @tehlaser
    @tehlaser Рік тому +29

    I had a regex blow up on me like that once. Not **quite** as silly as .*(?:.*=.*), but pretty close. The regex library we were using implemented backtracking with recursion, so instead of eating CPUs like a bag of chips it would instead masticate for a while before eventually running out of stack, whereupon it would puke Pringles. This was an especially fun one to fix because if you google “regex stack overflow” you’ll find that there are zillions of questions on stackoverflow about regexes that have nothing to do with stack overflows.
    And yes, in shame I must admit the regex in question did not fit on my screen all at once. In my defense, however, that was because a year or so earlier I had torn the line noise apart and put 3 to 5 characters of actual regex on each line, followed by a comment. Only two lines had // I have no idea, this shouldn’t do anything, but it doesn’t work without it.

    • @hentai824
      @hentai824 Рік тому +2

      lmaoo man i feel you

    • @tehlaser
      @tehlaser Рік тому +2

      Come to the dark side. We indent our regexes.

    • @yehoshuas.6917
      @yehoshuas.6917 5 місяців тому

      @@tehlaser jesus christ man

  • @BrandonCallender
    @BrandonCallender Рік тому +78

    I dont blame you for doing Cloudflare again, their RCAs are always excellent. This is such an excellent channel, you deserve far more subs! These are exceedingly entertaining and interesting for software engineers (and probably most other folks too!)

  • @RMDragon3
    @RMDragon3 Рік тому +10

    I kept wondering why they didn't just do a rollback to fix the issue, thanks for addressing that at the end.

    • @romannasuti25
      @romannasuti25 Рік тому

      Yeah, as much as I love Cloudflare for smaller stuff there's a reason a lot of large enterprises use Akamai. A little overpriced for a simple growth phase startup and not as transparent as Cloudflare when something breaks on their end, but that massive bucket list of features available with Ion Premier, Cloudlets, and many more, especially Datastreams and their web security analytics portal, is an absolute lifesaver. Hell, it helps us debug all sorts of broken stuff upstream of it too, although I wouldn't be surprised if Cloudflare offered something like Akamai Reference IDs for easy, enterprise-friendly tracing. Specifically, Akamai is really particular about having identical Staging and Production sections with really fast rollback when production error rates increase even a little.

    • @framegrace1
      @framegrace1 Рік тому

      All those rules are stored locally to each node, and you cannot rollback to a machine that is dead or so high on CPU that can't even handle a connection. I presume they globally disabled WAF and restarted the nodes, so when up, they didn't try to apply the WAF rules and were free to be rolled back/forward. Then they re-enabled WAF (very slowly, I presume :) ) and all was back to normal.

  • @D0Samp
    @D0Samp Рік тому +25

    The reason we predominantly use NFA regular expression engines is not just because they're usually faster if we don't throw non-degenerative expressions at them, but also because they support expressions that exceed the capabilities of a regular grammar, such as back references to a specific capture group that has been seen previously.

    • @MH_VOID
      @MH_VOID Рік тому +1

      I was under the impression that they're generally slower

    • @georgehelyar
      @georgehelyar Рік тому +2

      ​@@MH_VOIDfor a normal case the performance is generally similar, but the difference is that these linear engines like RE2 are more predictable and less likely to blow up in your face.
      If you don't have control of the pattern and the input, they are *much* safer, and losing features that depend on backtracking is generally not a big deal.
      If it's really performance critical just don't use regex at all if you can avoid it.

    • @teunmathijssen7459
      @teunmathijssen7459 Рік тому +2

      NFAs and DFAs are computationally equivalent and recognise only exactly the regular languages. So a NFA-backed RE engine would have to implement additional functionality as languages with backreferences are not regular.

    • @D0Samp
      @D0Samp Рік тому

      I've learned in the meantime that the biggest speed advantage is actually due to unrelated technologies such as a JIT compiler in PCRE2, which is in fact a top-down parser that happens to accept regex-like expressions. The only thing that is definitely faster about NFA is compiling regular expressions.

    • @paulstelian97
      @paulstelian97 Рік тому +1

      @@D0SampCertain types of things that regex engines/matchers support aren't true regex, and can't be covered by a DFA/NFA. That's a bit BS to me.

  • @CALEB94
    @CALEB94 Рік тому +7

    Awesome and informative video! A small correction: NFA matching is still linear in the input string. You just have to store the configuration as a set of NFA states, rather than a single state. You don't get exponentially many paths in the way you describe in the video because paths ending at the same state are merged in this set representation.

  • @LKRaider
    @LKRaider Рік тому +7

    I like the part of explaining non-capturing groups and then throwing them out the window immediately after

    • @yfakolh7154
      @yfakolh7154 Рік тому

      Except it's wrong. You do non-capturing for performance reasons, to consume characters in some group (this is simply parentheses syntax reuse). In this particular case this was so obviously wrong I can't imagine anyone familiar not to spot this, but in general you shouldn't capture what's not required after the match is done.

  • @mariadragonbreath7130
    @mariadragonbreath7130 10 місяців тому +1

    I stumbled across your videos yesterday, and i find them really entertaining and interesting to watch! Thank you explaining these topics in a clear way that even if I know nothing about regex or cloudfare, i can still follow along and understand the video :)

  • @krozareq
    @krozareq Рік тому +5

    When in doubt, implement your expression in a delayed loop so it doesn't murder everything.

    • @PinePizza
      @PinePizza Рік тому +2

      Even without being into coding I understand how this could work and it puzzles me why they didn't do this lol.

  • @EchoingRuby
    @EchoingRuby Рік тому +29

    I mean they could have just done .*?=.* but I guess RE2 is safer long-term. Still this screams "I don't understand regex, it's just magic to me" on the part of that developer.

    • @killingtimeitself
      @killingtimeitself 10 місяців тому

      which is fair honestly, regex is basically just magic and once you understand the syntax you dont question it's ways.
      though im surprised nowhere else along the development process was anybody concerned over it. Apparently nobody that looked over it had any idea what it was doing.

    • @jacksoncremean1664
      @jacksoncremean1664 Місяць тому

      The star means to match 0 or more, making it lazy with ? Is pointless

  • @aethese
    @aethese Рік тому +8

    Love these videos Kevin! Your amazing storytelling, editing, animations, and everything else comes together in an amazing way! Love watching every video you put out, keep it up :)

  • @molotov6844
    @molotov6844 Рік тому +2

    The Lemmino music really rounds this video off

  • @minimalist_zero
    @minimalist_zero Рік тому +3

    This is an incredible video. You took very complex and difficult to understand concepts and simplified it well. Well done.

  • @yaksher
    @yaksher Рік тому +4

    @11:50 You don't need a DFA to guarantee time linear in the input length because you don't actually need to follow every path, you just need to keep track of the set of states that the NFA is in. This does make it more expensive proportional to the size of the NFA, but it's still linear in the input length.

  • @SaltpeterTaffy
    @SaltpeterTaffy Рік тому +13

    "Which you may notice is not linear."
    This is one of those comp. sci. campfire horror story jumpscares.

    • @abebuckingham8198
      @abebuckingham8198 7 місяців тому

      Right? I just saw 22, 33, 44+1 and immediately thought "oh no". 😆

  • @patterntrader690
    @patterntrader690 Рік тому +6

    Your videos are amazing, keep going you are bound to blow up

  • @qm3ster
    @qm3ster Рік тому +4

    Non-capturing groups (unlike lookahead and lookbehind) do get included in the match result (think $0), they just don't create an additional sub-match.
    Eg at 6:35, that would match $0: $1: $2: while removing the ?: would make it $0: $1: $2: $3:

  • @Timi7007
    @Timi7007 Рік тому +2

    I love the graphics, depicting real processes very well but hilariously funny at the same time!

  • @TheRealKhepri
    @TheRealKhepri Рік тому +3

    I appreciate that the Australian Cloudflare is upside down at 4:25

  • @MogDog66
    @MogDog66 Рік тому +2

    2:37 I love that he checks the "delete master branch after merging" box

  • @RandomDeforge
    @RandomDeforge Рік тому +4

    out of the 10,000 times that this topic has been covered on UA-cam in this exact amount of detail,
    this is so far the most recent.
    kudos!

  • @thedoble
    @thedoble Рік тому +1

    The animation and comedic aspect of this video is great. Plus its explained extremely well. Nice

  • @spalt
    @spalt 4 місяці тому +1

    paging kevin fang, your services are needed once again

  • @jacobclark288
    @jacobclark288 Рік тому +1

    these videos are so well produced, the jokes in the imagery are so on point.

  • @ChineseKiwi
    @ChineseKiwi 4 місяці тому +5

    The process seems awfully familiar...but how......

  • @ballinlikebill8334
    @ballinlikebill8334 Рік тому +7

    love the style keep it up man

  • @quixadhal
    @quixadhal Рік тому +2

    Well, from the thumbnail image, the regexp (.*=.*) says "find the LARGEST chunk of text possible before a literal = sign, then find the largest chunk after it, including other = signs if they exist", and it will walk the entire chunk of data many times to ensure it gets ALL of them.
    They probably meant to do (.*?=.*?), which would have found the SMALLEST chunks of text around literal = signs, and would stop as soon as it found even a single = sign.

  • @rousseauramirez
    @rousseauramirez Рік тому +5

    I'm starting to hope another outage somewhere on the internet occurs just to see a Kevin Fang video with hundreds of explosion effects in my recommendations yet again

  • @0causton0
    @0causton0 Рік тому +1

    I absolutely lost it when I saw the "Cloudflare Analytics Dashboard" - and then remembered I use real software that has a dashboard that looks very similar, not even as pretty but grey 1998 vibes

  • @Namerson
    @Namerson Рік тому +1

    A rollback being a special case of a rollforward had me kekking heartily

  • @maddoggLP
    @maddoggLP Рік тому

    Didn't expect to hear about theoretical computer science (which is a subject I take this year) in this video but nice work. It's nice to see actual real-world usage of converting e-NFA to DFA's. I wish our prof would have included this video in his lecture...

  • @mangoodbad13
    @mangoodbad13 Рік тому

    Things exploding in your videos is the entire reason I wake up every morning, thank you friend, it's freakin hilarious

  • @BurnerWah
    @BurnerWah Рік тому +2

    This video has a pretty good explanation of regex engines TBH

  • @Mempler
    @Mempler Рік тому +2

    You just explained regex which i literally couldn't get the grasp of in literally 30s

  • @MP3Martin
    @MP3Martin Рік тому +1

    Cloudflare's free tier has the most value of any other free tier on the internet. They give you access to almost everything that the big companies have access to, just with certain limitations like max rule count. Amazing company

  • @rileyalden723
    @rileyalden723 Рік тому +2

    this video about shitty code explains regex, dfas and nfas better in like 5 minutes than my university formal languages course did in months

  • @creamyhorror
    @creamyhorror Рік тому

    I can't believe I've only just run across your content - it's really well done and humorous, you're going places!

  • @Steamrick
    @Steamrick Рік тому +3

    I'm a simple man. The upside down Cloudflare over Australia made me laugh. Thanks.

  • @Shimasen.
    @Shimasen. Рік тому

    Never have I ever expected to hear LEMMiNO soundtracks anywhere, glad to hear it from this channel!

  • @ThisIsAnAccount
    @ThisIsAnAccount Рік тому

    God I love watching these vids, I love the duality of high quality, digestible, information coupled with a nice sprinkling of "don't be a dipsh*t" commentary over issues and causation. Developer wise, nothing brings me more joy about my job than someone pointing out how much of an imbecile I *could* have been on that one day.

  • @Dardasha_Studios
    @Dardasha_Studios Рік тому

    You are a gem,
    thanks for the detailed information.
    You didn't only explain complex information, you also explained how WAF companies work.
    Thanks again.
    Salam!!

  • @ladyravendale1
    @ladyravendale1 Рік тому

    I love your videos on the internet blowing up. Perfect blend of programming, memes, and good graphics

  • @mattym8
    @mattym8 Рік тому

    Excellent vid. Couldn't have been shorter. Didn't need to be longer. Learned lots. You might've achieved perfection.

  • @BurzowySzczurek
    @BurzowySzczurek Рік тому +2

    Another nice dev store, with interesting storytelling, really enjoyable to watch. Thanks, we are waiting for more : )

  • @carro179
    @carro179 7 місяців тому

    I understood more with this video than in half a semester of languages and automatons

  • @45armageddon
    @45armageddon Рік тому +12

    0:36 oh australia

  • @AndersonPEM
    @AndersonPEM Рік тому +5

    8:46 what tool are you using here? Can I use it to visualize other programming languages' regex engine?

    • @GamBar64
      @GamBar64 9 місяців тому

      I was gonna ask the same thing

  • @CFEF44AB1399978B0011
    @CFEF44AB1399978B0011 Рік тому +4

    The entire world runs on regular expressions that were written in rage.

  • @Giuwu_
    @Giuwu_ Рік тому +9

    > Delete master after the pull request is merged
    LMAOO

  • @Vulcapyro
    @Vulcapyro Рік тому +9

    The line at the end could be revised; it isn't that "convoluted" regex should be avoided, in fact writing regex that appears more complicated often ends up being better in that it's more specific. Really bad regex happens when people want to write something quick and dirty that will consume all valid cases, but without considering constraints, which was part of the issue here. I know this is probably what you meant by saying convoluted, but one thing missing here is a bit talking about what you should consider writing instead, and people unfamiliar with good vs bad regex might not come away with the right idea.

    • @hentai824
      @hentai824 Рік тому

      another based regex enjoyer

  • @tlxyxl8524
    @tlxyxl8524 Рік тому +2

    While technically all NFA can be converted into DFA, the algorithm to do so (subset construction algorithm) has an exponential worst case time runtime. This is probably why people try to approximate the DFA.

  • @OlegDorbitt
    @OlegDorbitt Рік тому

    11:00 Wow, you've explained the usefulness of using DFA way better than my professor! Now it all makes sense!

  • @FackeYu
    @FackeYu Рік тому

    breuuugh, how has this channel not been suggested to me much sooner

  • @RSZA011
    @RSZA011 Рік тому

    this is by far my most interesting youtube channel . please keep it up ! I really enjoy this content

  • @seansull
    @seansull 9 місяців тому

    Dude your graphics are so funny 💀 I just about died when you edited the flow chart for WAF deployments

  • @tarvankrieken
    @tarvankrieken Рік тому +3

    I'm currently working on my thesis where I'm also dealing with regular expressions and their internal NFA representation quite a bit. And I recently encountered some papers about these risks of the naive backtracking implementation most engines use. Very interesting to now see one of these problems occur in practice. This happening in practice also gives quite some validity to the approach Rust is taking, which ensures good asymptotic characteristics.
    Only thing I want to point out is that you saying "the increase in steps can potentially be exponential" is fairly misleading. Since it makes it sound like this particular case has an exponential asymptotic runtime, while it only has a quadratic one.

    • @framegrace1
      @framegrace1 Рік тому

      Define "Most". All unix utilities use DFA's . The problem are interpreted languages, they use PCRE so they can use the ~= operator dynamically, as the recursive regex doesn't need compilation and the implementation is easier. (even bash uses PCRE for the =~ operator)
      Using compilation caching, for example, all of them could use the normal regex library, but I guess it increases complexity.

    • @tarvankrieken
      @tarvankrieken Рік тому

      @@framegrace1 That's fair. Most regex implementations I've encountered 🙂
      But I'm a windows user plus I mostly use higher level interpreted languages, so adds up with what you're saying.
      The DFA version combined with capture groups just gets much more complex, and I don't know of any implementation that supports lookarounds with this either yet.

  • @michaelhorne8366
    @michaelhorne8366 Рік тому

    While I appreciate you inverting the logo for Cloudflare Australia, our monitors are already upside down, you're good bro 👌

  • @kanal7523
    @kanal7523 Рік тому

    I love your videos and goofy animations please never stop doing these

  • @overreactengine
    @overreactengine Рік тому

    2:37 this tiny easy-to-miss detail right before the cut to the next shot killed me 😂

  • @hemal7551
    @hemal7551 Рік тому

    12:17 I really loved how you used sarcasm throughout the video😂
    You have perfect combination bro ngl

  • @Twisted_Code
    @Twisted_Code 5 місяців тому +1

    0:55 As soon as you said RegEx, I guessed it was a lazy match everything wildcard.

  • @Hi.Its.J
    @Hi.Its.J 11 місяців тому +1

    the upside down Australian Cloudflare server was a nice touch

  • @villan7h
    @villan7h Рік тому +3

    It MAY break the internet… approved 😊

  • @TheNinToaster
    @TheNinToaster Рік тому +1

    9:24 I love you how explain how NFAs work on a basic level, wish I had this for my required Theory of Computing class in my undergrad lmao

  • @Cyber_Chriis
    @Cyber_Chriis Рік тому +1

    Dude your illustrations are so good and funny! :D

  • @QuickQuips
    @QuickQuips Рік тому

    I really like how you described the newer forms of regex. They look like Moore/Meely machines which use 1s and 0s and are handy with digital design.

    • @abebuckingham8198
      @abebuckingham8198 7 місяців тому +1

      Mealy/Moore Machines are a kind of deterministic finite automata for a special case. So it's no coincidence they look the same.

  • @jlinkels
    @jlinkels Рік тому

    The contents is excellent, informative and clear. The narration is about 2 times too fast. OK, I am a boomer, but hey you are much more relaxed if you take a breath now and then.

  • @azmodanpc
    @azmodanpc Рік тому

    State machines are a bane of my college years. My professor could not explain it clearly and I got stuck on that exam.

  • @mcduffchannel
    @mcduffchannel Рік тому +1

    not me wildly trying to recall every regex i've written directly into prod and wondering if i've ever had a dirt eating moment😳

  • @dealloc
    @dealloc Рік тому +4

    If the input is arbitrary and especially comes from untrusted source, regex is almost never the answer, unless you have already limited the input size and complexity. This kind of logic should've called for a deterministic parser instead. While it would possibly have taken longer to write, it would have been more readable. On top of that, I would also consider using property-based testing framework to test with arbitrary input.
    Good learning experience, and great that Cloudflare shared detailed explanation of their mistakes.

    • @framegrace1
      @framegrace1 Рік тому +1

      You cannot (practically) use a deterministic parser for WAF, but you can compile the requests as said in the video which is what they should have done instead of using PCRE (I bet they used PCRE because it doesn't need compilation and "makes things easier")

  • @lydierayn
    @lydierayn 10 місяців тому

    6:49 That Lemmino's Cypher as a background music slaps my dude

  • @Game_Hi
    @Game_Hi Рік тому +1

    Thanks, I’ll remember that next time working on regex for the whole internet

  • @sairao4492
    @sairao4492 Рік тому +1

    This is a great video. You made it very easy to follow. Took me back to my computational models class.

  • @Danielo515
    @Danielo515 10 місяців тому

    This is the best explanation of the incident that I saw. Good job

  • @dasherreal
    @dasherreal Рік тому

    This channel concept is brilliant. Thank you so much.