Merge pull request #3991 from lonvia/interpolation-on-addresses

Add support for addr:interpolation on housenumbers

add support for expanding interpolations on housenumbers

Merge pull request #3923 from kad-link/ci/windows-smoke

CI: add dev-only Windows smoke workflow

Merge pull request #3989 from lonvia/rework-misc-tests

More production SQL use in unit tests

Merge pull request #3988 from jayaddison/pr-3957-followup/empty-name-field

Indexing: fixup: add presence check for hstore 'name' field

Indexing: fixup: add presence check for hstore `name` field

make database import unit tests against real SQL

use original tables for database check tests

Merge pull request #3987 from lonvia/rework-postcode-tests

Rework postcode unit tests

replace postcode mock with standard postcode table fixtures

ci/windows: install osm2pgsql binary and enable full unit tests suite

ci/windows: enable full python unit test setup for windows

ci/windows: add Postgresql setup action to tests

implement stage 1 : python unit tests

Add experimental Windows CI workflow

Merge pull request #3986 from lonvia/rework-tiger-tests

Rework unit tests for import of tiger data

Merge pull request #3985 from lonvia/rework-indexing-test

Rework unit tests for indexing

rework tiger unit tests to use production pgsql functions

pass reverse-only as a separate parameter

... instead of hacking the internal structures of SQLPreprocessor

rewrite indexing tests to use standard table fixtures

add fixtures for postcode, interpolation table creation and filling

Merge pull request #3984 from lonvia/avoid-custom-table-definition-in-tests

Reuse table creation SQL in unittest

use load_sql fixture instead of explicit SQLPreprocessor

reorganise fixtures for placex table

Now follows the same pattern as fixtures for other tables and
uses the production SQL for table creation.

simplify property test table implementation

switch table definitions in conftest to use production SQL

add fixtures for country table

add function for inserting data to testing cursor

use better SQL quoting in test cursor implementation

fix potential cancelling race with full queue

Merge pull request #3982 from lonvia/split-table-creation

Split up table creation SQL

split up table creation SQL into separate files

remove unused indexes and sequences

Merge pull request #3957 from jayaddison/issue-2714/linked-places-default-language

Indexing: add default-language placename from linked places

Merge pull request #3979 from jayaddison/issue-2714-prep/extract-rank-zero-specialcasing

Indexer: relocate zero-ranked-address indexing

Merge pull request #3980 from lonvia/security-smells

Improve SQL query assembly

Indexer: relocate zero-ranked-address indexing

ignore tables with odd names in SQLPreprocessor

avoid f-strings in SQL creation in tests

use psycopg.sql for SQL building in tokenizer

only allow alphanumeric and dash in DATABASE_WEBUSER

This variable is used a lot in raw SQL. Avoid injection issues.

sanity check class names before inserting into classtype tables

The subsequent INSERT is done on an unqouted table name, making in
theory an SQL injection through an OSM value possible. In practise
this cannot happen because we check for the existance of the table.
During the creation of the classtype tables there is a sanity
check in place to disallow any table names that consist of anything
other than alphanumeric characters.

Merge pull request #3978 from jayaddison/issue-2714-prep/index-boundaries-method-signature-nitpick

Refactor: add default params to Indexer.index_boundaries

Merge pull request #3975 from kad-link/fix/utf8-encoding-clean

Fix: Enforce explicit UTF-8 encoding in file I/O

Refactor: add default params to `index_boundaries`

fix: add utf-8 encoding in read-write files

Indexing: invert boolean logic to factor-out empty `ELSE` clause

Relates-to commit fa2a789e270a716be8655ce75ba5ed37fba9e09d.

Merge pull request #3970 from lonvia/improve-dev-docs

Some minor improvement to developer docs

Indexing: manage the case where no default-language exists

Relates-to commit 6fee784c9fc2bf6ca9428482713347258161e0cc.

Merge pull request #3960 from jayaddison/tests/has-pending-monkeypatch-robustness

Tests: parameter-agnostic 'Indexer.has_pending' monkeypatching

point to developer docs from CONTRIBUTING.md

docs: fix list of packages for development install

Indexing: add default-language placename from linked places

Tests: parameter-agnostic `has_pending` monkeypatching

Instead of relying on runtime parameter compatibility between
the patched `has_pending` method and `list.pop`, use a proxy
lambda function that accepts arbitrary keyword params.

Merge pull request #3966 from remo-lab/fix/sql-injection-truncate

Fix SQL injection in truncate_data_tables

Fix SQL injection in truncate_data_tables

Signed-off-by: remo-lab <remopanda7@gmail.com>

Merge pull request #3962 from lonvia/docs-deploy

Docs: switch deployment to use gunicorn's asgi/uwsgi support

docs: switch deployment to use gunicorn's asgi/uwsgi support

Merge pull request #3949 from Itz-Agasta/try

Feat: Add admin function for granting access to read-only user

Merge pull request #3959 from Aditya30ag/fix/readme-nominatim-api-module-path

Fix README: update Nominatim API server module path

Fix README: update Nominatim API server module path

Merge pull request #3955 from AmmarYasser455/fix/typos

docs: fix multiple typos in documentation and source code

Merge pull request #3952 from jayaddison/pr-3687-followup/boundary-admin-level-for-linkage

Tests: resolve an issue in the place-linkage name expansion test case

Tests: set boundary admin level to enable linking

Preserve import error tables during freeze

- Remove 'import_polygon_%' from UPDATE_TABLES to keep import_polygon_error
and import_polygon_delete tables in frozen databases.

- These tables contain permanent import error tracking data and should not
be deleted during freeze. The ro-access grant system expects them to exist
in all database states.

Restore grants for dynamic tables in tokenizer, migration, and tiger import

docs: fix multiple typos in documentation and source code

Merge pull request #3951 from Itz-Agasta/cli

Feat: Adds layer filtering option to search cli command

Updates layer selection to allow optional default

- Modifies layer argument handling to permit no default layers appropriate.
- Update the help text for the layer parameter in the reverse command

Merge pull request #3950 from jayaddison/fixup/sql-debug-output-escaping

Fixup: add single-quote escaping within debug message

Adds layer filtering option to search command

Introduces a cli argument to restrict search results
to specified data layers, enabling more targeted queries.

Fixup: add single-quote escaping within debug message

Adds mock grants SQL file for import test

Adds option to grant web user read-only DB access

Introduces a command-line flag to grant read-only access to the web user for all tables, improving ease of permissions management during refresh operations.

Grants read-only access after import

Adds execution of grant statements to provide read-only privileges
for the web user following table creation or via a dedicated function.
Facilitates easier post-import permission management.

Moves db grant statements to dedicated script

Centralizes all read-only access grants into a single SQL script, ensuring permissions are managed in one place.

Merge pull request #3943 from Itz-Agasta/test_fix

Tests: Replace eval() with ast.literal_eval() for safer parsing

Switch to  ast.literal_eval for dict parsing

Due to  some test data in the BDD feature files includes Python raw strings and escape sequences that standard json.loads() cannot parse switching to safer Python literal evaluation
for converting string representations of dictionaries.

Replaces eval with json.loads for safer dict parsing

Switches from eval to json.loads when parsing string representations
of dictionaries to  prevent arbitrary code
execution.

Merge pull request #3946 from lonvia/enable-entrances-for-reverse

Enable entrance lookup for reverse and lookup

Merge pull request #3906 from AyushDharDubey/fix/issue_2463-Use-search_name-table-for-TIGER-data-imports-on-'dropped'-databases

Use `search_name` as fallback for TIGER imports when update tables are dropped

Merge pull request #3945 from lonvia/fix-starlette-tests

Update Starlette tests to using their TestClient

enable entrances for reverse and lookup

update Starlette tests to using their TestClient

fix mypy typing for Starlette state object (#3944)

Merge pull request #3939 from lonvia/more-table-constraints

Add NOT NULL and UNIQUE contraints on tables

update test frozen db: new tiger import mechanism

permit import of tiger after freeze

Add fallback search mechanism for dropped databases lookup

do not attempt to delete old data for newly created placex entries

add NOT NULL and UNIQUE constraints where possible

ignore countries without geometry or country code for location_area

do not insert entries with empty name into search name

ensure correct indexed_status transitions

make sure that importance is always set to a non-null value

Secondary importance might return invalid values in some cases.

make sure array_merge() never returns null

immediately terminate indexing when a task catches an exception

BDD: make sure randomly generated names always contain a letter

Add PR template (#3934)

Merge pull request #3930 from lonvia/remove-new-query-log-table

Remove unused new_query_log table