When I enter an item filter query into the text-box in the items tab, I have to click "Apply filter". This is slow, and I'd prefer to press the Enter key as it's faster. I'm running Firefox 85.0 on Ubuntu 20.04.

For reference, the element in question is here:

Investigate building and shipping an Electron version of Data Explorer

Sometimes user will see the following error:

But the underlying exception is:
"Microsoft.Azure.Documents.DocumentClientException: The authorization token is not valid at the current time. Please create another token and retryMicrosoft.Azure.Documents.DocumentClientException: The authorization token is not valid at the current time. Please create another token and retry"

Currently any permission error is bucketed as a firewall issue. In this case, the user to spend time trying to "fix" their firewall or ask CSS why firewall isn't working. The exception should be caught separately.

@languy We appear to be pulling in two different versions of monaco-editor:

➜ cosmos-explorer git:(master) npm ls monaco-editor
[email protected] /home/steve/cosmos-explorer
├─┬ @azure/[email protected]
│ └── [email protected] deduped
├─┬ @nteract/[email protected]
│ └── [email protected]
└── [email protected]

All the bundles with monaco contain both. Can you see if we can figure out how to just pull in one?

https://github.com/Azure/cosmos-explorer/blob/master/src/Main.tsx#L278

A list of tasks broken down by phase but not in any particular order:

Pre-release

• Remove wrapper directories
• Remove PowerShell scripts or make cross-platform
• Commit snapshot of code to repo
• Ensure LICENSE file exists
• Ensure CONTRIBUTING file exists
• Open source this repo

Post-release

• Setup CI

When running 'npm install', I get a sym link error for canvas. What installation instructions do you have for resolving this?

https://github.com/Azure/cosmos-explorer/blob/master/src/Main.tsx#L275

When a user toggles "Unlimited" it the code is actually setting the value to 100 page max size:

Backend team has signed off on removing autopilot v2 (preview) code from the create path. Still waiting to see if we can remove from scale/settings path.

FYI @victor-meng @srinathnarayanan @gitanuj in case you touch any of that code

Data explorer allows creation of containers with unique keys but they are not displayed post creation in any of the container settings pages.

Anywhere we are using the JS SDK to perform control plane operations, we need to switch to Cosmos Resource Provider when logged in as an AAD user. I generated a preliminary list of all the operations that need to be checked:

• Update Collection
• Update Offer
• Delete Collection
• Delete Database
• Read Collections
• Read Databases
• Read Collection
• Read Collection Quota Info
• Read Offer
• Read Offers
• Read Databases
• Get or Create Database and Collection
• Create Database
• Refresh Cached Offers
• Refresh Cached Resources
• Read Subcription
• Read Subscription Defaults

Each of these will also need possibly support a fallback to the JS SDK for connection string users.

Frequently we see errors logged in the notifications bad that look like "{}". This is because we are using JSON.stringify(error) in many places. Instances of Error have no enumerable properties which is the reason for this behavior. We should add a custom linting rule or type changes to prohibit using JSON.stringify(error).

I tried this and there were ~20 failures. We need to investigate and finish this upgrade.

@gitanuj We're getting flagged by GitHub for having PATs in source code https://github.com/Azure/cosmos-explorer/security/secret-scanning

Can we come up with a plan to move these out of source code? They could become an environment variable in tests. Or we could use something like dotenv https://www.npmjs.com/package/dotenv. Let me know if you have other ideas.

Its a bit strange to have the unit tests hit a live endpoint. Could we mock them or use something like nock to record and mock them? https://github.com/nock/nock#recording

1. Create a Mongo collection that triggers an error. Example: Try creating a mongo collection with a shard key /id
2. Get back a cryptic response.text() is not a function error message

The issue is this line here:

The catch clause has an implicit any type on response. But in this case, it is not an instance of Response. It is a RP specific error format:

TypeScript 4.0 has a fix for this https://devblogs.microsoft.com/typescript/announcing-typescript-4-0-beta/#unknown-on-catch

cc @vchske

The auth header gets set as undefined in the offer request. Works fine in Chrome and Edge

How to repro:

• Have more than 100 vertices in the graph
• At least one vertex has an id that contains $. For example: foo$bar.
• Execute g.V() and see the ScriptError coming out of the backend.

The problem:
Double-quoted strings denote Groovy string interpolation where ‘$’ denotes the beginning of a placeholder.
Eg.

"${cosmosAccount}" -- this is expected to be replaced with the value of variable cosmosAccount.

To avoid this and treat the entire string as a literal, single-quoted strings should be used.
Eg.
'$Default'

So the fix would be to ensure that the gremlin result pager uses single quoted strings for partition key/id values in the query builder.

In azure portal, trying to add row to existing Cassandra table. Getting error but not sure what the error is.

Our actions can run pretty long. We can use this action to cancel them when another commit is pushed on the same branch https://github.com/styfle/cancel-workflow-action

In the latest emulator build, there is a 405 error reported in the browser dev tools for:
https://localhost:8081/_explorer/undefined/api/disallowedLocations

@shmotlan This looks like a portal API call. Any ideas?

Bug repros every time in MPAC. Works correctly in PROD.

From initial investigation looks like something goes wrong when sending message to parent window to refresh caches.

#33 covers Mongo and SQL, but we need to add the other APIs as well

https://github.com/Azure/cosmos-explorer/blob/master/src/Main.tsx#L277

Azure Static Web Apps has the ability to deploy PRs from GitHub: https://azure.microsoft.com/en-us/services/app-service/static/

It would be awesome to use this functionality to deploy a stand-alone version of DE explorer on every push.

We've had a few cases of an errant console.log checked in to production. We should lint for these.

Tables collections with sparse data will sometimes error when paging through results. The code that is responsible for adding new headers is not working correctly and the table is rebuilt with no data in the UI.

https://github.com/Azure/cosmos-explorer/blob/master/src/Main.tsx#L282

@languy and I had a chat today about how we make better progress on migrating to React, strict mode TypeScript, and full ESlint. Incremental progress has been challenging due to how highly coupled our code is. There is coupling in code, but also between files, that makes incremental change hard. We brainstormed some changes that we think will help decrease coupling:

• Remove ViewModels where there is only one implementation
• Split up the ViewModels.ts file
• Remove OpenActionsStubs and replace with a jest mock for explorer
• Migrated the entire Hosted explorer wrapper to React
• Migrate the entire Emulator wrapper to React
• Remove Sinon (Lower priority. May not reduce coupling)
• Split up DataModels.ts file

https://github.com/Azure/cosmos-explorer/blob/master/src/Main.tsx#L279

@languy pointed out that we lost the nice UI for reporting test coverage when we moved to GitHub. It's not something I personally miss, but if others liked it we could look at options. There are some GitHub actions that will do this in a comment as well as some third party tools like https://github.com/apps/codecov that would do this.

1. Open a query tab
2. Open a stored procedure
3. Execute the stored procedure
4. Go back to query tab
5. You will not be able to edit the query

Suspect there is some issue here with multiple copies of Monaco on the page. Will take some further investigation

https://github.com/actions/cache

Good candidates for caching:

• node_modules
• cypress
• babel/typescript loader cache for webpack

Cosmos DB has a feature that allows you to set a custom path to be used in place of '_ts' when deciding whether to delete an item via TTL. As of now, setting and viewing the TimeToLivePropertyPath is possible through the SDK only.

https://docs.microsoft.com/en-us/dotnet/api/microsoft.azure.cosmos.containerproperties.timetolivepropertypath?view=azure-dotnet#definition

The following example has been the cause of a few recent support tickets:

Contoso Dev Team has two developers: Alex and Bob.
1. Alex sets TimeToLivePropertyPath to a customer property “/myTimestamp”
2. Bob goes to use Cosmos DB later and can’t figure out why items aren’t being deleted based on TTL
3. Bob doesn’t realize that Alex had set a TimeToLivePropertyPath. Bob is confused and sees this as unexplainable behavior

Being able to view and modify the TimeToLivePropertyPath in the Portal would help decrease confusion here.

Most terminals support linked text via this package: https://www.npmjs.com/package/terminal-link

Reported via ICM 214687554:

Data is not showing up with a filter is passed. Tested in Azure storage explorer , Portal and cosmos.azure.com , all of them show similar behavior, data is shown accurately when there is no filter, whenever a filter is passed results are not accurate , 1st page doesn't load , navigating to other page shows some data.

Can the subscription list be alpha ordered?

#33 adds support for E2E tests that use connection string. We should also add AAD based tests, but doing this might be a little tricky. One idea is to use a dummy iframe that simulates the portal. The AAD token can be stored in GitHub and injected into the parent dummy iframe to simulate the experience of a logged-in AAD user

https://github.com/Azure/cosmos-explorer/blob/master/src/Main.tsx#L280

When you start the Cosmos DB Emulator with a custom key using the /Key or /KeyFile command line parameters it seems impossible to use the Explorer as it has no way to provide the key to use. It defaults to the built-in key.

I am running the emulator on my host machine and am developing my application in a Docker container. Therefore I need to enable network access on the emulator, which requires a custom key to be provided. I can successfully connect from the container to the emulator on the host using the generated key, however I am not able to inspect the data using the Explorer on the host.

Instead, code should use config.platform. There are still a few places we use the old window.dataExplorerPlatform

https://github.com/Azure/cosmos-explorer/blob/master/src/Main.tsx#L281

https://github.com/Azure/cosmos-explorer/blob/master/src/Main.tsx#L274

From @languy: It appears that the indexing setting is (incorrectly) ignored for Graph and SQL API when the user is signed in with AAD. Under these conditions, the default settings are being used (automatic indexing), so choosing indexing OFF will produce the same result as indexing set to Automatic.

https://github.com/Azure/cosmos-explorer/blob/master/src/Main.tsx#L283

https://github.com/Azure/cosmos-explorer/blob/master/src/Main.tsx#L276

Describe the bug
The following queries work as expected:

SELECT VALUE (true)
SELECT VALUE (1)
SELECT VALUE ("foo")

But the following queries return an error "Cannot set property 'headers' of undefined":

SELECT VALUE (false)
SELECT VALUE (0)
SELECT VALUE ("")
SELECT VALUE (null)

To Reproduce
Run the queries in the Data Explorer.

Expected behavior
The correct values being returned without error.

Actual behavior
The above error is raised.