Use new endpoint for project creation (#1202)

hramezani · web-flow · commit 964f585b73e7 · 2025-06-30T18:09:41.000+02:00
diff --git a/logfire/_internal/config.py b/logfire/_internal/config.py
@@ -1624,7 +1624,7 @@ def create_new_project(
                     default=project_name_default,
                 )
 
-            url = urljoin(logfire_api_url, f'/v1/projects/{organization}')
+            url = urljoin(logfire_api_url, f'/v1/organizations/{organization}/projects')
             try:
                 response = session.post(url, headers=headers, json={'project_name': project_name})
                 if response.status_code == 409:
diff --git a/tests/test_cli.py b/tests/test_cli.py
@@ -521,7 +521,7 @@ def test_projects_new_with_project_name_and_org(
             }
         }
         m.post(
-            'https://logfire-us.pydantic.dev/v1/projects/fake_org',
+            'https://logfire-us.pydantic.dev/v1/organizations/fake_org/projects',
             [create_project_response],
         )
 
@@ -562,7 +562,7 @@ def test_projects_new_with_project_name_without_org(
             }
         }
         m.post(
-            'https://logfire-us.pydantic.dev/v1/projects/fake_org',
+            'https://logfire-us.pydantic.dev/v1/organizations/fake_org/projects',
             [create_project_response],
         )
 
@@ -605,7 +605,7 @@ def test_projects_new_with_project_name_and_wrong_org(
             }
         }
         m.post(
-            'https://logfire-us.pydantic.dev/v1/projects/fake_org',
+            'https://logfire-us.pydantic.dev/v1/organizations/fake_org/projects',
             [create_project_response],
         )
 
@@ -646,7 +646,7 @@ def test_projects_new_with_project_name_and_default_org(
             }
         }
         m.post(
-            'https://logfire-us.pydantic.dev/v1/projects/fake_org',
+            'https://logfire-us.pydantic.dev/v1/organizations/fake_org/projects',
             [create_project_response],
         )
 
@@ -693,7 +693,7 @@ def test_projects_new_with_project_name_multiple_organizations(
             }
         }
         m.post(
-            'https://logfire-us.pydantic.dev/v1/projects/fake_org',
+            'https://logfire-us.pydantic.dev/v1/organizations/fake_org/projects',
             [create_project_response],
         )
 
@@ -747,7 +747,7 @@ def test_projects_new_with_project_name_and_default_org_multiple_organizations(
             }
         }
         m.post(
-            'https://logfire-us.pydantic.dev/v1/projects/fake_default_org',
+            'https://logfire-us.pydantic.dev/v1/organizations/fake_default_org/projects',
             [create_project_response],
         )
 
@@ -786,7 +786,7 @@ def test_projects_new_without_project_name(
             }
         }
         m.post(
-            'https://logfire-us.pydantic.dev/v1/projects/fake_org',
+            'https://logfire-us.pydantic.dev/v1/organizations/fake_org/projects',
             [create_project_response],
         )
 
@@ -829,7 +829,7 @@ def test_projects_new_invalid_project_name(
             }
         }
         m.post(
-            'https://logfire-us.pydantic.dev/v1/projects/fake_org',
+            'https://logfire-us.pydantic.dev/v1/organizations/fake_org/projects',
             [create_project_response],
         )
 
@@ -877,7 +877,7 @@ def test_projects_new_error(tmp_dir_cwd: Path, default_credentials: Path) -> Non
             }
         }
         m.post(
-            'https://logfire-us.pydantic.dev/v1/projects/fake_org',
+            'https://logfire-us.pydantic.dev/v1/organizations/fake_org/projects',
             [create_project_response],
         )
 
@@ -910,7 +910,7 @@ def test_projects_without_project_name_without_org(
             }
         }
         m.post(
-            'https://logfire-us.pydantic.dev/v1/projects/fake_org',
+            'https://logfire-us.pydantic.dev/v1/organizations/fake_org/projects',
             [create_project_response],
         )
 
@@ -985,7 +985,7 @@ def test_projects_new_create_project_error(tmp_dir_cwd: Path, default_credential
         stack.enter_context(m)
         m.get('https://logfire-us.pydantic.dev/v1/projects/', json=[])
         m.get('https://logfire-us.pydantic.dev/v1/organizations/', json=[{'organization_name': 'fake_org'}])
-        m.post('https://logfire-us.pydantic.dev/v1/projects/fake_org', text='Error', status_code=500)
+        m.post('https://logfire-us.pydantic.dev/v1/organizations/fake_org/projects', text='Error', status_code=500)
 
         with pytest.raises(LogfireConfigError, match='Error creating new project.'):
             main(['projects', 'new', 'myproject', '--org', 'fake_org'])
diff --git a/tests/test_configure.py b/tests/test_configure.py
@@ -925,7 +925,9 @@ def test_initialize_project_use_existing_project_no_projects(tmp_dir_cwd: Path,
                 'project_url': 'fake_project_url',
             }
         }
-        request_mocker.post('https://logfire-api.pydantic.dev/v1/projects/fake_org', [create_project_response])
+        request_mocker.post(
+            'https://logfire-api.pydantic.dev/v1/organizations/fake_org/projects', [create_project_response]
+        )
 
         logfire.configure(send_to_logfire=True)
         wait_for_check_token_thread()
@@ -1023,7 +1025,9 @@ def test_initialize_project_not_using_existing_project(
                 'project_url': 'fake_project_url',
             }
         }
-        request_mocker.post('https://logfire-api.pydantic.dev/v1/projects/fake_org', [create_project_response])
+        request_mocker.post(
+            'https://logfire-api.pydantic.dev/v1/organizations/fake_org/projects', [create_project_response]
+        )
         request_mocker.post(
             'https://logfire-api.pydantic.dev/v1/organizations/fake_org/projects/fake_project/write-tokens/',
             [create_project_response],
@@ -1145,7 +1149,7 @@ def test_initialize_project_create_project(tmp_dir_cwd: Path, tmp_path: Path, ca
             }
         }
         request_mocker.post(
-            'https://logfire-api.pydantic.dev/v1/projects/fake_org',
+            'https://logfire-api.pydantic.dev/v1/organizations/fake_org/projects',
             [
                 create_existing_project_response,
                 create_reserved_project_response,
@@ -1241,7 +1245,7 @@ def test_initialize_project_create_project_default_organization(tmp_dir_cwd: Pat
             }
         }
         request_mocker.post(
-            'https://logfire-api.pydantic.dev/v1/projects/fake_org',
+            'https://logfire-api.pydantic.dev/v1/organizations/fake_org/projects',
             [create_project_response],
         )
 

Original file line number	Diff line number	Diff line change
`@@ -1624,7 +1624,7 @@ def create_new_project(`
`1624`	`1624`	`default=project_name_default,`
`1625`	`1625`	`)`
`1626`	`1626`
`1627`		`- url = urljoin(logfire_api_url, f'/v1/projects/{organization}')`
	`1627`	`+ url = urljoin(logfire_api_url, f'/v1/organizations/{organization}/projects')`
`1628`	`1628`	`try:`
`1629`	`1629`	`response = session.post(url, headers=headers, json={'project_name': project_name})`
`1630`	`1630`	`if response.status_code == 409:`
Original file line number	Diff line number	Diff line change
`@@ -521,7 +521,7 @@ def test_projects_new_with_project_name_and_org(`
`521`	`521`	`}`
`522`	`522`	`}`
`523`	`523`	`m.post(`
`524`		`- 'https://logfire-us.pydantic.dev/v1/projects/fake_org',`
	`524`	`+ 'https://logfire-us.pydantic.dev/v1/organizations/fake_org/projects',`
`525`	`525`	`[create_project_response],`
`526`	`526`	`)`
`527`	`527`
`@@ -562,7 +562,7 @@ def test_projects_new_with_project_name_without_org(`
`562`	`562`	`}`
`563`	`563`	`}`
`564`	`564`	`m.post(`
`565`		`- 'https://logfire-us.pydantic.dev/v1/projects/fake_org',`
	`565`	`+ 'https://logfire-us.pydantic.dev/v1/organizations/fake_org/projects',`
`566`	`566`	`[create_project_response],`
`567`	`567`	`)`
`568`	`568`
`@@ -605,7 +605,7 @@ def test_projects_new_with_project_name_and_wrong_org(`
`605`	`605`	`}`
`606`	`606`	`}`
`607`	`607`	`m.post(`
`608`		`- 'https://logfire-us.pydantic.dev/v1/projects/fake_org',`
	`608`	`+ 'https://logfire-us.pydantic.dev/v1/organizations/fake_org/projects',`
`609`	`609`	`[create_project_response],`
`610`	`610`	`)`
`611`	`611`
`@@ -646,7 +646,7 @@ def test_projects_new_with_project_name_and_default_org(`
`646`	`646`	`}`
`647`	`647`	`}`
`648`	`648`	`m.post(`
`649`		`- 'https://logfire-us.pydantic.dev/v1/projects/fake_org',`
	`649`	`+ 'https://logfire-us.pydantic.dev/v1/organizations/fake_org/projects',`
`650`	`650`	`[create_project_response],`
`651`	`651`	`)`
`652`	`652`
`@@ -693,7 +693,7 @@ def test_projects_new_with_project_name_multiple_organizations(`
`693`	`693`	`}`
`694`	`694`	`}`
`695`	`695`	`m.post(`
`696`		`- 'https://logfire-us.pydantic.dev/v1/projects/fake_org',`
	`696`	`+ 'https://logfire-us.pydantic.dev/v1/organizations/fake_org/projects',`
`697`	`697`	`[create_project_response],`
`698`	`698`	`)`
`699`	`699`
`@@ -747,7 +747,7 @@ def test_projects_new_with_project_name_and_default_org_multiple_organizations(`
`747`	`747`	`}`
`748`	`748`	`}`
`749`	`749`	`m.post(`
`750`		`- 'https://logfire-us.pydantic.dev/v1/projects/fake_default_org',`
	`750`	`+ 'https://logfire-us.pydantic.dev/v1/organizations/fake_default_org/projects',`
`751`	`751`	`[create_project_response],`
`752`	`752`	`)`
`753`	`753`
`@@ -786,7 +786,7 @@ def test_projects_new_without_project_name(`
`786`	`786`	`}`
`787`	`787`	`}`
`788`	`788`	`m.post(`
`789`		`- 'https://logfire-us.pydantic.dev/v1/projects/fake_org',`
	`789`	`+ 'https://logfire-us.pydantic.dev/v1/organizations/fake_org/projects',`
`790`	`790`	`[create_project_response],`
`791`	`791`	`)`
`792`	`792`
`@@ -829,7 +829,7 @@ def test_projects_new_invalid_project_name(`
`829`	`829`	`}`
`830`	`830`	`}`
`831`	`831`	`m.post(`
`832`		`- 'https://logfire-us.pydantic.dev/v1/projects/fake_org',`
	`832`	`+ 'https://logfire-us.pydantic.dev/v1/organizations/fake_org/projects',`
`833`	`833`	`[create_project_response],`
`834`	`834`	`)`
`835`	`835`
`@@ -877,7 +877,7 @@ def test_projects_new_error(tmp_dir_cwd: Path, default_credentials: Path) -> Non`
`877`	`877`	`}`
`878`	`878`	`}`
`879`	`879`	`m.post(`
`880`		`- 'https://logfire-us.pydantic.dev/v1/projects/fake_org',`
	`880`	`+ 'https://logfire-us.pydantic.dev/v1/organizations/fake_org/projects',`
`881`	`881`	`[create_project_response],`
`882`	`882`	`)`
`883`	`883`
`@@ -910,7 +910,7 @@ def test_projects_without_project_name_without_org(`
`910`	`910`	`}`
`911`	`911`	`}`
`912`	`912`	`m.post(`
`913`		`- 'https://logfire-us.pydantic.dev/v1/projects/fake_org',`
	`913`	`+ 'https://logfire-us.pydantic.dev/v1/organizations/fake_org/projects',`
`914`	`914`	`[create_project_response],`
`915`	`915`	`)`
`916`	`916`
`@@ -985,7 +985,7 @@ def test_projects_new_create_project_error(tmp_dir_cwd: Path, default_credential`
`985`	`985`	`stack.enter_context(m)`
`986`	`986`	`m.get('https://logfire-us.pydantic.dev/v1/projects/', json=[])`
`987`	`987`	`m.get('https://logfire-us.pydantic.dev/v1/organizations/', json=[{'organization_name': 'fake_org'}])`
`988`		`- m.post('https://logfire-us.pydantic.dev/v1/projects/fake_org', text='Error', status_code=500)`
	`988`	`+ m.post('https://logfire-us.pydantic.dev/v1/organizations/fake_org/projects', text='Error', status_code=500)`
`989`	`989`
`990`	`990`	`with pytest.raises(LogfireConfigError, match='Error creating new project.'):`
`991`	`991`	`main(['projects', 'new', 'myproject', '--org', 'fake_org'])`
Original file line number	Diff line number	Diff line change
`@@ -925,7 +925,9 @@ def test_initialize_project_use_existing_project_no_projects(tmp_dir_cwd: Path,`
`925`	`925`	`'project_url': 'fake_project_url',`
`926`	`926`	`}`
`927`	`927`	`}`
`928`		`- request_mocker.post('https://logfire-api.pydantic.dev/v1/projects/fake_org', [create_project_response])`
	`928`	`+ request_mocker.post(`
	`929`	`+ 'https://logfire-api.pydantic.dev/v1/organizations/fake_org/projects', [create_project_response]`
	`930`	`+ )`
`929`	`931`
`930`	`932`	`logfire.configure(send_to_logfire=True)`
`931`	`933`	`wait_for_check_token_thread()`
`@@ -1023,7 +1025,9 @@ def test_initialize_project_not_using_existing_project(`
`1023`	`1025`	`'project_url': 'fake_project_url',`
`1024`	`1026`	`}`
`1025`	`1027`	`}`
`1026`		`- request_mocker.post('https://logfire-api.pydantic.dev/v1/projects/fake_org', [create_project_response])`
	`1028`	`+ request_mocker.post(`
	`1029`	`+ 'https://logfire-api.pydantic.dev/v1/organizations/fake_org/projects', [create_project_response]`
	`1030`	`+ )`
`1027`	`1031`	`request_mocker.post(`
`1028`	`1032`	`'https://logfire-api.pydantic.dev/v1/organizations/fake_org/projects/fake_project/write-tokens/',`
`1029`	`1033`	`[create_project_response],`
`@@ -1145,7 +1149,7 @@ def test_initialize_project_create_project(tmp_dir_cwd: Path, tmp_path: Path, ca`
`1145`	`1149`	`}`
`1146`	`1150`	`}`
`1147`	`1151`	`request_mocker.post(`
`1148`		`- 'https://logfire-api.pydantic.dev/v1/projects/fake_org',`
	`1152`	`+ 'https://logfire-api.pydantic.dev/v1/organizations/fake_org/projects',`
`1149`	`1153`	`[`
`1150`	`1154`	`create_existing_project_response,`
`1151`	`1155`	`create_reserved_project_response,`
`@@ -1241,7 +1245,7 @@ def test_initialize_project_create_project_default_organization(tmp_dir_cwd: Pat`
`1241`	`1245`	`}`
`1242`	`1246`	`}`
`1243`	`1247`	`request_mocker.post(`
`1244`		`- 'https://logfire-api.pydantic.dev/v1/projects/fake_org',`
	`1248`	`+ 'https://logfire-api.pydantic.dev/v1/organizations/fake_org/projects',`
`1245`	`1249`	`[create_project_response],`
`1246`	`1250`	`)`
`1247`	`1251`