Skip to content

Explicitly listen only on localhost / 127.0.0.1 when launching the server #880

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 1 commit into from
Aug 7, 2025
Merged

Explicitly listen only on localhost / 127.0.0.1 when launching the server #880

merged 1 commit into from
Aug 7, 2025

Conversation

greschd
Copy link
Member

@greschd greschd commented Aug 6, 2025

  • In direct launch mode, listen only on localhost
  • In docker_compose launch mode, limit connections to coming from 127.0.0.1 by limiting the source IP in the ports mapping

@codecov Codecov
Copy link

codecov bot commented Aug 6, 2025 •
edited
Loading

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 94.96%. Comparing base (95c0bea) to head (b55ccd0).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files 
@@           Coverage Diff           @@
##             main     #880   +/-   ##
=======================================
  Coverage   94.96%   94.96%           
=======================================
  Files         110      110           
  Lines        5758     5758           
  Branches      293      293           
=======================================
  Hits         5468     5468           
  Misses        289      289           
  Partials        1        1
Flag Coverage Δ
direct-launch 92.18% <ø> (ø)
minimum-deps 93.06% <ø> (ø)
python-3.10 94.14% <ø> (ø)
python-3.11 94.07% <ø> (ø)
python-3.12 94.07% <ø> (ø)
python-3.13 94.32% <ø> (ø)
server-2024R2 90.48% <ø> (ø)
server-2025R1 94.11% <ø> (ø)
server-2025R2 94.07% <ø> (ø)
server-2026R1 94.07% <ø> (ø)
server-latest 94.72% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@greschd greschd merged commit 9fc6f14 into main Aug 7, 2025
34 checks passed
@greschd greschd deleted the fix/improve_grpc_server_url_defaults branch August 7, 2025 11:33
greschd added a commit that referenced this pull request Aug 11, 2025
@greschd @dependabot
Release 0.2.1 (#882)
1a63ce1 
* Bump hypothesis from 6.131.21 to 6.131.28 in the dependencies group (#857)

Bumps the dependencies group with 1 update: [hypothesis](https://github.com/HypothesisWorks/hypothesis).


Updates `hypothesis` from 6.131.21 to 6.131.28
- [Release notes](https://github.com/HypothesisWorks/hypothesis/releases)
- [Commits](HypothesisWorks/hypothesis@hypothesis-python-6.131.21...hypothesis-python-6.131.28)

---
updated-dependencies:
- dependency-name: hypothesis
  dependency-version: 6.131.28
  dependency-type: direct:development
  update-type: version-update:semver-patch
  dependency-group: dependencies
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump the dependencies group with 4 updates (#859)

* Bump the dependencies group with 4 updates

Bumps the dependencies group with 4 updates: [ansys-mechanical-core](https://github.com/ansys/pymechanical), [mypy](https://github.com/python/mypy), [ansys-sphinx-theme](https://github.com/ansys/ansys-sphinx-theme) and [hypothesis](https://github.com/HypothesisWorks/hypothesis).


Updates `ansys-mechanical-core` from 0.11.16 to 0.11.18
- [Release notes](https://github.com/ansys/pymechanical/releases)
- [Changelog](https://github.com/ansys/pymechanical/blob/main/CHANGELOG.md)
- [Commits](ansys/pymechanical@v0.11.16...v0.11.18)

Updates `mypy` from 1.15.0 to 1.16.0
- [Changelog](https://github.com/python/mypy/blob/master/CHANGELOG.md)
- [Commits](python/mypy@v1.15.0...v1.16.0)

Updates `ansys-sphinx-theme` from 1.4.4 to 1.5.0
- [Release notes](https://github.com/ansys/ansys-sphinx-theme/releases)
- [Commits](ansys/ansys-sphinx-theme@v1.4.4...v1.5.0)

Updates `hypothesis` from 6.131.28 to 6.132.0
- [Release notes](https://github.com/HypothesisWorks/hypothesis/releases)
- [Commits](HypothesisWorks/hypothesis@hypothesis-python-6.131.28...hypothesis-python-6.132.0)

---
updated-dependencies:
- dependency-name: ansys-mechanical-core
  dependency-version: 0.11.18
  dependency-type: direct:production
  update-type: version-update:semver-patch
  dependency-group: dependencies
- dependency-name: mypy
  dependency-version: 1.16.0
  dependency-type: direct:development
  update-type: version-update:semver-minor
  dependency-group: dependencies
- dependency-name: ansys-sphinx-theme
  dependency-version: 1.5.0
  dependency-type: direct:development
  update-type: version-update:semver-minor
  dependency-group: dependencies
- dependency-name: hypothesis
  dependency-version: 6.132.0
  dependency-type: direct:development
  update-type: version-update:semver-minor
  dependency-group: dependencies
...

Signed-off-by: dependabot[bot] <support@github.com>

* Ignore override error in analysis ply test method

---------

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Dominik Gresch <dominik.gresch@ansys.com>

* Add a `connect_local` launch mode (#860)

- Adds a new launch mode `connect_local`, which connects to an
  existing ACP server, but without using the file transfer
  service.
- Fixes `ACPInstance.models`: The `Model._from_object_info` was
  incorrectly called with a bare gRPC channel, instead of a
  `ServerWrapper` instance.
  This bug wasn't caught because it only affects the case
  when connecting to an existing server which already has models
  loaded. Otherwise, the instance caching mechanism means that
  all models are instantiated when initially loaded via
  `import_model`, which works correctly.

* Bump the version to 0.3.0.dev0 (#856)

Bump the development version to `0.3.0.dev0`, since the `main` 
branch now works towards the next minor release.

* Bump ansys/actions from 9 to 10 (#861)

Bumps [ansys/actions](https://github.com/ansys/actions) from 9 to 10.
- [Release notes](https://github.com/ansys/actions/releases)
- [Changelog](https://github.com/ansys/actions/blob/main/CHANGELOG.md)
- [Commits](ansys/actions@v9...v10)

---
updated-dependencies:
- dependency-name: ansys/actions
  dependency-version: '10'
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump pyvista/setup-headless-display-action from 4.1 to 4.2 (#862)

Bumps [pyvista/setup-headless-display-action](https://github.com/pyvista/setup-headless-display-action) from 4.1 to 4.2.
- [Release notes](https://github.com/pyvista/setup-headless-display-action/releases)
- [Commits](pyvista/setup-headless-display-action@f744438...7d84ae8)

---
updated-dependencies:
- dependency-name: pyvista/setup-headless-display-action
  dependency-version: '4.2'
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump the dependencies group with 3 updates (#863)

Bumps the dependencies group with 3 updates: [ansys-sphinx-theme](https://github.com/ansys/ansys-sphinx-theme), [pytest](https://github.com/pytest-dev/pytest) and [hypothesis](https://github.com/HypothesisWorks/hypothesis).


Updates `ansys-sphinx-theme` from 1.5.0 to 1.5.2
- [Release notes](https://github.com/ansys/ansys-sphinx-theme/releases)
- [Commits](ansys/ansys-sphinx-theme@v1.5.0...v1.5.2)

Updates `pytest` from 8.3.5 to 8.4.0
- [Release notes](https://github.com/pytest-dev/pytest/releases)
- [Changelog](https://github.com/pytest-dev/pytest/blob/main/CHANGELOG.rst)
- [Commits](pytest-dev/pytest@8.3.5...8.4.0)

Updates `hypothesis` from 6.132.0 to 6.135.3
- [Release notes](https://github.com/HypothesisWorks/hypothesis/releases)
- [Commits](HypothesisWorks/hypothesis@hypothesis-python-6.132.0...hypothesis-python-6.135.3)

---
updated-dependencies:
- dependency-name: ansys-sphinx-theme
  dependency-version: 1.5.2
  dependency-type: direct:development
  update-type: version-update:semver-patch
  dependency-group: dependencies
- dependency-name: pytest
  dependency-version: 8.4.0
  dependency-type: direct:development
  update-type: version-update:semver-minor
  dependency-group: dependencies
- dependency-name: hypothesis
  dependency-version: 6.135.3
  dependency-type: direct:development
  update-type: version-update:semver-minor
  dependency-group: dependencies
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump the dependencies group with 5 updates (#864)

Bumps the dependencies group with 5 updates:

| Package | From | To |
| --- | --- | --- |
| [typing-extensions](https://github.com/python/typing_extensions) | `4.13.2` | `4.14.0` |
| [ansys-tools-path](https://github.com/ansys/ansys-tools-path) | `0.7.1` | `0.7.2` |
| [pytest-cov](https://github.com/pytest-dev/pytest-cov) | `6.1.1` | `6.2.1` |
| [pytest-cases](https://github.com/smarie/python-pytest-cases) | `3.8.6` | `3.9.1` |
| [hypothesis](https://github.com/HypothesisWorks/hypothesis) | `6.135.3` | `6.135.10` |


Updates `typing-extensions` from 4.13.2 to 4.14.0
- [Release notes](https://github.com/python/typing_extensions/releases)
- [Changelog](https://github.com/python/typing_extensions/blob/main/CHANGELOG.md)
- [Commits](python/typing_extensions@4.13.2...4.14.0)

Updates `ansys-tools-path` from 0.7.1 to 0.7.2
- [Release notes](https://github.com/ansys/ansys-tools-path/releases)
- [Changelog](https://github.com/ansys/ansys-tools-path/blob/v0.7.2/CHANGELOG.md)
- [Commits](ansys/ansys-tools-path@v0.7.1...v0.7.2)

Updates `pytest-cov` from 6.1.1 to 6.2.1
- [Changelog](https://github.com/pytest-dev/pytest-cov/blob/master/CHANGELOG.rst)
- [Commits](pytest-dev/pytest-cov@v6.1.1...v6.2.1)

Updates `pytest-cases` from 3.8.6 to 3.9.1
- [Release notes](https://github.com/smarie/python-pytest-cases/releases)
- [Changelog](https://github.com/smarie/python-pytest-cases/blob/main/docs/changelog.md)
- [Commits](smarie/python-pytest-cases@3.8.6...3.9.1)

Updates `hypothesis` from 6.135.3 to 6.135.10
- [Release notes](https://github.com/HypothesisWorks/hypothesis/releases)
- [Commits](HypothesisWorks/hypothesis@hypothesis-python-6.135.3...hypothesis-python-6.135.10)

---
updated-dependencies:
- dependency-name: typing-extensions
  dependency-version: 4.14.0
  dependency-type: direct:production
  update-type: version-update:semver-minor
  dependency-group: dependencies
- dependency-name: ansys-tools-path
  dependency-version: 0.7.2
  dependency-type: direct:production
  update-type: version-update:semver-patch
  dependency-group: dependencies
- dependency-name: pytest-cov
  dependency-version: 6.2.1
  dependency-type: direct:development
  update-type: version-update:semver-minor
  dependency-group: dependencies
- dependency-name: pytest-cases
  dependency-version: 3.9.1
  dependency-type: direct:development
  update-type: version-update:semver-minor
  dependency-group: dependencies
- dependency-name: hypothesis
  dependency-version: 6.135.10
  dependency-type: direct:development
  update-type: version-update:semver-patch
  dependency-group: dependencies
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump the dependencies group with 5 updates (#866)

---
updated-dependencies:
- dependency-name: ansys-tools-path
  dependency-version: 0.7.3
  dependency-type: direct:production
  update-type: version-update:semver-patch
  dependency-group: dependencies
- dependency-name: ansys-mechanical-core
  dependency-version: 0.11.19
  dependency-type: direct:production
  update-type: version-update:semver-patch
  dependency-group: dependencies
- dependency-name: mypy
  dependency-version: 1.16.1
  dependency-type: direct:development
  update-type: version-update:semver-patch
  dependency-group: dependencies
- dependency-name: pytest
  dependency-version: 8.4.1
  dependency-type: direct:development
  update-type: version-update:semver-patch
  dependency-group: dependencies
- dependency-name: hypothesis
  dependency-version: 6.135.14
  dependency-type: direct:development
  update-type: version-update:semver-patch
  dependency-group: dependencies
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Update poetry lockfile (#868)

* Update poetry lockfile
* Exclude numpydoc 1.9.0

* Explicitly listen only on localhost / 127.0.0.1 when launching the server (#880)

- In `direct` launch mode, listen only on `localhost`
- In `docker_compose` launch mode, limit connections to coming from `127.0.0.1` by limiting the source IP in the `ports` mapping

* Allow stackup thickness and area weight to be None (#873)

When the fabrics in a stackup do have a material, or the stackup
has no fabrics, the 'thickness' and 'area_weight' properties
cannot be computed. This change allows these properties to be
'None' in these cases.
This case is only supported in the 26.1 server version or later.
Earlier versions would raise an error when getting the properties
under these conditions.

* Bump version to 0.2.1

---------

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
greschd added a commit that referenced this pull request Aug 12, 2025
@greschd
Explicitly listen only on localhost / 127.0.0.1 when launching the se…
71711dc 
…rver (#880)

- In `direct` launch mode, listen only on `localhost`
- In `docker_compose` launch mode, limit connections to coming from `127.0.0.1` by limiting the source IP in the `ports` mapping
@greschd greschd mentioned this pull request Aug 12, 2025
Merged
greschd added a commit that referenced this pull request Aug 12, 2025
@greschd
Merge pull request #885 from ansys/fix/adapt_to_service_pack
e9621b2 
Release 0.1.1

- Backport #808, #880, #884 to the `0.1` release.
- Update the poetry lockfile.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@SRaimondi SRaimondi SRaimondi approved these changes

@roosre roosre roosre approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@greschd @SRaimondi @roosre