Skip to content
Permalink

Comparing changes

Choose two branches to see what’s changed or to start a new pull request. If you need to, you can also or learn more about diff comparisons.

Open a pull request

Create a new pull request by comparing changes across two branches. If you need to, you can also . Learn more about diff comparisons here.
base repository: temporalio/temporal
Failed to load repositories. Confirm that selected base ref is valid, then try again.
Loading
base: v1.22.6
Choose a base ref
...
head repository: temporalio/temporal
Failed to load repositories. Confirm that selected head ref is valid, then try again.
Loading
compare: v1.22.7
Choose a head ref
  • 3 commits
  • 7 files changed
  • 2 contributors

Commits on Mar 28, 2024

  1. Add validation for a few string fields (#5487)

    Add string validation for a few string fields
    
    Since we disable utf8 string validation from proto level, we want to
    enforce minimal validation for some key fields.
    
    Unit tests
    
    No
    
    <!-- Have you made sure this change doesn't falsify anything currently
    stated in `docs/`? If significant
    new behavior is added, have you described that in `docs/`? -->
    
    <!-- Is this PR a hotfix candidate or does it require a notification to
    be sent to the broader community? (Yes/No) -->
    yiminc authored and tdeebswihart committed Mar 28, 2024
    Copy the full SHA
    2099dfd View commit details
  2. Validate normal TQ on sticky poll (#5488)

    ## What changed?
    Validate normal TQ on sticky poll
    
    ## Why?
    On sticky poll, we fetch normal TQ's UserData. So an invalid normal TQ
    on a sticky queue would still trigger a load for the invalid normal
    queue.
    
    ## How did you test it?
    Manual test with invalid TQ.
    
    ## Potential risks
    <!-- Assuming the worst case, what can be broken when deploying this
    change to production? -->
    
    ## Documentation
    <!-- Have you made sure this change doesn't falsify anything currently
    stated in `docs/`? If significant
    new behavior is added, have you described that in `docs/`? -->
    
    ## Is hotfix candidate?
    <!-- Is this PR a hotfix candidate or does it require a notification to
    be sent to the broader community? (Yes/No) -->
    yiminc authored and tdeebswihart committed Mar 28, 2024
    Copy the full SHA
    627e2d2 View commit details
  3. Bump Server version to 1.22.7

    temporal-data committed Mar 28, 2024
    Copy the full SHA
    1c396c1 View commit details
Loading