Regression: CreateLinkPostRequestBody serializes retainInheritedPermissions=false by default starting in 1.59.2
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 75/100
Línea de trabajo
Comienza con el punto de entrada generado CreateLinkPostRequestBody que se muestra en la ruta de importación y reproduce la serialización usando el ejemplo test.py. Compara la salida con las versiones 1.58.0 y 1.59.2, centrándote en los valores omitidos de retainInheritedPermissions y sendNotification. Se considera terminado cuando las propiedades no establecidas ya no serialicen valores false, permitiendo que se apliquen los valores predeterminados de la API createLink.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Describe the bug
Context
I was using the CreateLinkPostRequestBody to create some link to my driveItems, and it was working properly, but at some point I noticed that some of these files would not be visible to some users anymore (inherited permissions were lost).
The "bug"
Starting with msgraph-sdk-python 1.59.2, CreateLinkPostRequestBody serializes: { "retainInheritedPermissions": false, "sendNotification": false } when these properties are not explicitly set by the caller.
This changes the behavior of the createLink API. According to the Microsoft Graph documentation, retainInheritedPermissions is optional and the service default when omitted is true .
Expected behavior
I believe that the CreateLinkPostRequestBody default attribute values should follow the defaults of the createLink API.
Thus setting retainInheritedPermissions to true (or unset) for the createLink API if not provided in CreateLinkPostRequestBody.
How to reproduce
test.py
from msgraph.generated.drives.item.items.item.create_link.create_link_post_request_body import CreateLinkPostRequestBody
request_body = CreateLinkPostRequestBody(
type="view",
scope="organization"
)
from kiota_serialization_json.json_serialization_writer import JsonSerializationWriter
writer = JsonSerializationWriter()
request_body.serialize(writer)
print(writer.get_serialized_content().decode('utf-8'))
Results
version 1.58.0
{
"scope": "organization",
"type": "view"
}
This will let the createLink API default the retainInheritedPermissions to True.
version 1.59.2 and 1.60.0
{
"retainInheritedPermissions": false,
"scope": "organization",
"sendNotification": false,
"type": "view"
}
This will remove existing permissions.
SDK Version
1.59.2
Latest version known to work for scenario above?
1.58.0
Known Workarounds
Explicitly adding the retain_inherited_permissions = True
request_body = CreateLinkPostRequestBody(
retain_inherited_permissions=True,
type="view",
scope="organization"
)
Debug output
Click to expand log
```</details>
### Configuration
Ubuntu 22.04
x86_64
### Other information
_No response_
- Lenguaje dominante
- Python
- Estrellas
- 630
- Forks
- 96
- Merge medio
- 15 h 20 min
- PR fusionados (30 d)
- 3
Guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de microsoftgraph/msgraph-sdk-python
-
status:waiting-for-triage type:bug
Dificultad 1/5 Menos de una hora Aptitud para principiantes 90/100
microsoftgraph/msgraph-sdk-python#1570 ·
-
Needs: Attention :wave: type:bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 62/100
microsoftgraph/msgraph-sdk-python#1287 · 2 comentarios · 1 reacción ·
-
status:waiting-for-triage type:bug
Dificultad 3/5 1-2 días Aptitud para principiantes 52/100
microsoftgraph/msgraph-sdk-python#1578 ·
-
Batch Request Support Abiertostatus:waiting-for-triage
Dificultad 5/5 Más de una semana Aptitud para principiantes 25/100
microsoftgraph/msgraph-sdk-python#1557 ·
-
status:waiting-for-triage type:bug
Dificultad 3/5 1-2 días Aptitud para principiantes 55/100
microsoftgraph/msgraph-sdk-python#1556 · 1 comentario · 1 reacción ·
Todos los issues de microsoftgraph/msgraph-sdk-python
Issues similares
-
Add: hunch Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 74/100
AbdelStark/awesome-typesafe#104 ·
-
enhancement
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
DiamondLightSource/dodal#2211 ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 88/100
openml/openml-python#1749 ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 88/100
sipyourdrink-ltd/bernstein#6191 ·