Находки в опенсорсе: Python
985 subscribers
4 photos
221 links
Легкие задачки в опенсорсе из мира Python

Чат: @opensource_findings_chat
Download Telegram
🚀 New issue to wemake-services/django-modern-rest by @sobolevn
📝 JWT token blacklist app: design conditional apps (#462)


We need to design how we can add conditional django apps that have models to the framework.
For example, right now we can add an app to blacklist some jwt tokens.

What's interesting about that?

1. We need to design this app in a way that it is not required by default, with no visible parts
2. We need to design it that it will be very easy to install and use. For example: it should provide a mixin class that can be used together with JWTSyncAuth / JWTAsyncAuth and a pre-existing common subclass

So, usage will look something like:

from django_modern_rest.some.place.for.app.models import BlacklistedJWTToken

class BlacklistMixin:
def check_auth(self, user: 'AbstractBaseUser', token: JWTToken) -> None:
super().check_user()
if BlacklistedJWTToken.objects.filter(jti=token.jti).exists():
# Token was blacklisted:
raise NotAuthenticatedError

We use jti to identify tokens, as other jwt's do.
It is now possible after #461

Sources of inspiration:

https://github.com/jazzband/djangorestframework-simplejwt/blob/master/rest_framework_simplejwt/token_blacklist/models.py
https://django-rest-framework-simplejwt.readthedocs.io/en/latest/blacklist_app.html

#enhancement #good_first_issue #help_wanted #django_modern_rest
sent via relator
🚀 New issue to wemake-services/django-modern-rest by @sobolevn
📝 Add `cookies` support to `APIError` (#465)


Right now we support cookies in our API:

django-modern-rest/django_modern_rest/response.py

Lines 78 to 85 in ddfe907

But, APIError does not allow to set any cookies:

django-modern-rest/django_modern_rest/response.py

Lines 64 to 75 in ddfe907

This is unfortunate. We need to be able to do anything we can do with a regular response with APIError.

This is a really easy issue for first-timers, welcome :)


#enhancement #good_first_issue #help_wanted #django_modern_rest
sent via relator
🚀 New issue to wemake-services/django-modern-rest by @sobolevn
📝 Add unit tests for pagination (#484)


Right now we only test pagination in the doctests: https://github.com/wemake-services/django-modern-rest/blob/master/docs/examples/integrations/pagination.py

Let's add the same test to test_unit as well. Create new test_pagination folder and put it there.

This is a very easy task for entry-level controbutors :)


#enhancement #good_first_issue #help_wanted #django_modern_rest
sent via relator
🚀 New issue to wemake-services/wemake-python-styleguide by @sobolevn
📝 `WPS617` false positive with keyword args (#3596)


self._media_by_precedence = sorted(
(
media_type
for parser in self._parsers.values()
if (media_type := MediaType(parser.content_type)).quality != 0
),
key=lambda media: (media.specificity, media.quality), # noqa: WPS617
reverse=True,
)

It says:

django_modern_rest/negotiation.py

52:17 WPS617 Found lambda assigned as an attribute
key=lambda media: (media.specificity, media.quality),
^

Which is clearly a bug, it is a named param, not an attribute.
PRs are welcome, this is easy to fix.


#bug #help_wanted #levelstarter #good_first_issue #wemake_python_styleguide #wps
sent via relator
🚀 New issue to wemake-services/django-modern-rest by @kondratevdev
📝 Enable `positive_data_acceptance` in `schemathesis` (#496)


At the moment, this check is disabled in the configuration.

We need to enable it (delete the line, because by default we have everything enabled) and fix the error that this check catches.


#enhancement #help_wanted #django_modern_rest
sent via relator
🚀 New issue to wemake-services/django-modern-rest by @sobolevn
📝 Add an option to rename schemas (#505)


Right now schemas are registered like regular python classes, their name is just type(...).__name__ we need a way to rename schemas in the openapi.

I propose adding __dmr_schema_name__ special attribute to do that.

This will also require a test case for the openapi.


#enhancement #good_first_issue #help_wanted #django_modern_rest
sent via relator
🚀 New issue to wemake-services/wemake-python-styleguide by @sobolevn
📝 Remove `WPS354` (#3601)


This rules has more problems that it solves:

1. Two yields and yield from are two different things
2. Async iterators cannot use yield from, but WPS354 still raises even in async def

So, it is time to remove this rule. I was wrong about that :(
See other rules with disabled_since attribute on how to disable a rule.


#feature #help_wanted #levelstarter #good_first_issue #wemake_python_styleguide #wps
sent via relator
🫡1
🚀 New issue to ag2ai/faststream by @yann-combarnous
📝 Bug: AsyncAPI documentation fails when Confluent uses oauth bearer authentication (#2774)


Describe the bug
When using Confluent Kafka with oauthbearer authentication, the generated AsyncAPI schema is:

 "securitySchemes":{"oauthbearer":{"type":"oauth2","$ref":""}}

An empty $ref: "" causes the AsyncAPI React component to attempt file resolution via readFile. This is a FastStream bug in schema generation.

How to reproduce
Include source code:

from faststream.confluent import KafkaBroker

broker = KafkaBroker(
config={ ...config... },
security=SASLOAuthBearer(use_ssl=True)
)
...

Expected behavior
AsyncAPI documentation is correctly generated.

Observed behavior
It fails with a file cannot be read error.

Screenshots
If applicable, attach screenshots to help illustrate the problem.

Environment
Running FastStream 0.6.5 with CPython 3.13.11 on Darwin

Additional context


#bug #good_first_issue #faststream #ag2ai
sent via relator
🚀 New issue to wemake-services/django-modern-rest by @milssky
📝 Improving XML processing in `XMLParser` for correct validation of Pydantic models (#541)


The current implementation of XMLParser in django_test_app (and the example in the documentation) leads to massive validation errors (400 Bad Request) during automated testing via Schemathesis. The problem is caused by two factors:

1. Extra nesting (Root Element): xmltodict.parse always returns a dictionary where the data is nested in the key corresponding to the name of the root tag. Pydantic models (for example, _RequestModel) expects the data to be "flat", which is why validation crashes with the field required error for all fields, since they are wrapped in the tag name.
2. Type mismatch for empty tags: XML does not have built-in types. Empty tags () are parsed by xmltodict as None. If the Pydantic model expects str, it causes an error validation, although in the context of XML, an empty tag is often equivalent to an empty string.

Most likely, this bug ( #499 ) is related to this, but the problem may be complex.


#enhancement #help_wanted #django_modern_rest
sent via relator
🚀 New issue to wemake-services/django-modern-rest by @sobolevn
📝 Add `auth=` parameter to `sse()` function (#546)


I forgot to add auth= to parameters of

django-modern-rest/dmr/sse/builder.py

Lines 63 to 79 in 8866e4c

This is totally required. Docs and tests are also required :)


#enhancement #good_first_issue #help_wanted #django_modern_rest
sent via relator
🚀 New issue to wemake-services/wemake-python-styleguide by @sobolevn
📝 `WPS457` false positive with `while` in `try` / `except` (#3603)


I have the following code:

def factory() -> Iterator[bytes]:
try:
while True: # noqa: WPS457
yield async_to_sync(async_anext)(iterator)
except StopAsyncIteration:
pass

It should not raise WPS457, because while is in try / except.
It can clearly raise and we handle the error. So, it is not infinite.

PRs are welcome! 👍


#bug #help_wanted #levelstarter #good_first_issue #wemake_python_styleguide #wps
sent via relator
🚀 New issue to ag2ai/faststream by @lesnik512
📝 Feature: Static membership for aiokafka broker (group_instance_id) (#2782)


Hi!

Seems like aiokafka broker missing group_instance_id argument which required for static membership to work.

https://aiokafka.readthedocs.io/en/stable/api.html

group_instance_id (str or None) – name of the group instance ID used for static membership (KIP-345)

For confluent kafka it exists #2606


#enhancement #good_first_issue #faststream #ag2ai
sent via relator
🚀 New issue to wemake-services/django-modern-rest by @sobolevn
📝 Support `etag` (#572)


Docs: https://developer.mozilla.org/en-US/docs/Web/HTTP/Reference/Headers/ETag
Implementation example: https://docs.litestar.dev/latest/usage/responses.html#etag

We already can set etag as a header in @modify with NewHeader.
What do we need exactly?

1. We need ETag dataclass to represent the parsed header. It should also know how to unparse itself into a header
2. We need docs about sending ETag header and getting header If-Match back

#enhancement #good_first_issue #help_wanted #django_modern_rest
sent via relator
🚀 New issue to wemake-services/django-modern-rest by @sobolevn
📝 Add `Settings.semantic_responses` flag (#589)


Users might not need our semantic responses at all. It needs a kill switch.

We need to add Settings.semantic_responses flag, which should be True by default.
If set to False, we should not add any extra response specs to the final metadata.

Probably, we should add if check for this in EndpointMetadata.response_spec_providers

This would need several tests:

1. Snapshot test for the resulting OpenAPI spec
2. Test that response validation would still be required, missing responses must raise 419

Also, this would require docs in configuration.rst


#enhancement #good_first_issue #help_wanted #django_modern_rest
sent via relator
1
🚀 New issue to wemake-services/django-modern-rest by @sobolevn
📝 Add 3 more tests for component schema generation (#603)


There are several cases that we don't cover in tests/test_unit/test_openapi/test_schema_snapshots.py

1. We need to test Query component schema generation with one field in __dmr_force_list__ and one other regular field

 django-modern-rest/tests/test_unit/test_openapi/test_schema_snapshots.py

 Line 66 in 8555cd5

2. We need to test Query component with at least one field in

 django-modern-rest/tests/test_unit/test_openapi/test_schema_snapshots.py

 Line 67 in 8555cd5

These corner cases can just be added to the existing _QueryModel

1. We also need to test at least one file field as a list, add one field to __dmr_force_list__ for FileMetadata component as well

 django-modern-rest/tests/test_unit/test_openapi/test_schema_snapshots.py

 Line 122 in 8555cd5

This can be added to _SeveralFiles model.

This is a super easy issue for first-timers only :)


#enhancement #good_first_issue #help_wanted #python #django_modern_rest
sent via relator
🚀 New issue to wemake-services/django-modern-rest by @sobolevn
📝 Make an `@overload`ed function from `load_schema` (#625)


Currently load_schema can accept just a single argument if should_generate_examples is False.
Otherwise it needs extra params:

django-modern-rest/dmr/openapi/mappers/schema_loader.py

Lines 21 to 28 in 6c06cd9

This needs to be an overload.
This is a very easy task :)


#enhancement #good_first_issue #help_wanted #django_modern_rest
sent via relator
🚀 New issue to wemake-services/django-modern-rest by @sobolevn
📝 Support django's `i18n` for translation (#426)


All our user-facing texts must use django's i18n.
We will only provide en-us translation for now. Later more can be added if needed.


#enhancement #good_first_issue #help_wanted #django_modern_rest
sent via relator