Erik Cederstrand

Results 206 comments of Erik Cederstrand

Yes. See https://github.com/puiterwijk/flask-oidc/issues/147#issuecomment-1207160732

I think this is connected to the issue reported in https://github.com/ecederstrand/exchangelib/issues/1273. I don't have access to a test account with public folders, so I would suggest to debug this as...

Ok. Adding a breakpoint in `BaseFolder.children()` and stepping through the code is your best bet in debugging this. If you [enable debug logging](https://ecederstrand.github.io/exchangelib/#troubleshooting) you should be able to see the...

Actually, there's a slight possibility that https://github.com/ecederstrand/exchangelib/commit/d9035d03960797f9277381845e4ec98f837798ea fixes this. Can you give it a try?

@dtalkachou Did you find a solution to this? Is it apparent in all exchangelib versions, or is there a certain one that exhibits the error?

Hi @iglimanaj, Thanks for the additional information! Interesting that `_folders_map` does contain the public folders. This must mean that the parent folder ID of the child folders does not match...

Weird. We ask for the `ParentFolderId` field when getting subfolders. If you [enable debug logging](https://ecederstrand.github.io/exchangelib/#troubleshooting) and check the request and response XML, you should be able to see whether we...

`Folder.parent` is a property, defined here: https://github.com/ecederstrand/exchangelib/blob/c310c65e7b4eb12e2ba8037a589794f9eb668b51/exchangelib/folders/base.py#L869 It ends up calling `RootOfHierarchy.get_folder()`, defined here: https://github.com/ecederstrand/exchangelib/blob/c310c65e7b4eb12e2ba8037a589794f9eb668b51/exchangelib/folders/roots.py#L72 which looks up by ID in `_folders_map`. You'll either need to trace the code into...

Ah, maybe what happens is that `parent_folder_id` for the public folder points to the root folder of the account owning the folder, not the `public_folders_root` of the current account. That...

Available as https://github.com/ecederstrand/exchangelib/pull/1316