Thanks to visit codestin.com
Credit goes to github.com

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: saltstack-formulas/nginx-formula
Failed to load repositories. Confirm that selected base ref is valid, then try again.
Loading
base: v2.7.1
Choose a base ref
...
head repository: saltstack-formulas/nginx-formula
Failed to load repositories. Confirm that selected head ref is valid, then try again.
Loading
compare: v2.7.2
Choose a head ref
  • 5 commits
  • 10 files changed
  • 4 contributors

Commits on May 20, 2021

  1. chore(pre-commit): use info report level for rstcheck [skip ci] 

    * Automated using myii/ssf-formula#325
    @myii
    myii committed May 20, 2021
    Configuration menu
    Copy the full SHA
    a55f741 View commit details
    Browse the repository at this point in the history

Commits on Jun 14, 2021

  1. fix(snippets): ignore servers or snippets when undefined 

    nginx.servers_config wants a lightened copy of the nginx map
to render as json; but, when it was trying to remove the
servers and snippets keys from the map it assumed their presence,
causing a KeyError if they were not present by its use of .pop().

While wrapping these in an "if" clause would likely be more
correct, along with replacing .pop() with del (if jinja even
supports that) the simplest change here is to just specify a
default value for .pop(), which obviates the KeyError.

Fixes #274
    @myii
    Jonathon Anderson authored and myii committed Jun 14, 2021
    Configuration menu
    Copy the full SHA
    6cb486d View commit details
    Browse the repository at this point in the history

  2. fix(certificates): ensure openssl installed before cmd.run 

    Fix failure highlighted on OpenSUSE Tumbleweed, where the `cmd.run` runs
before the `pkg.installed`:

* https://gitlab.com/saltstack-formulas/nginx-formula/-/jobs/1345325819#L2830
    @myii
    myii committed Jun 14, 2021
    Configuration menu
    Copy the full SHA
    0cd7c7b View commit details
    Browse the repository at this point in the history

  3. Merge pull request #275 from anderbubble/issues/274 

    Ignore missing servers or snippets when removing from map
    @javierbertoli
    javierbertoli authored Jun 14, 2021
    Configuration menu
    Copy the full SHA
    c55120a View commit details
    Browse the repository at this point in the history

  4. chore(release): 2.7.2 [skip ci] 

    ## [2.7.2](v2.7.1...v2.7.2) (2021-06-14)

### Bug Fixes

* **certificates:** ensure `openssl` installed before `cmd.run` ([0cd7c7b](0cd7c7b)), closes [/gitlab.com/saltstack-formulas/nginx-formula/-/jobs/1345325819#L2830](https://github.com//gitlab.com/saltstack-formulas/nginx-formula/-/jobs/1345325819/issues/L2830)
* **snippets:** ignore servers or snippets when undefined ([6cb486d](6cb486d)), closes [#274](#274)
    @semantic-release-bot
    semantic-release-bot committed Jun 14, 2021
    Configuration menu
    Copy the full SHA
    b19fdde View commit details
    Browse the repository at this point in the history
Loading