Remove improperly applied flattening extension.

This checklist is used to make sure that common issues in a pull request are addressed. This will expedite the process of getting your pull request merged and avoid extra work on your part to fix issues discovered during the review process.
PR information

[ ] The title of the PR is clear and informative.
[ ] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For information on cleaning up the commits in your pull request, see this page.
[ ] Except for special cases involving multiple contributors, the PR is started from a fork of the main repository, not a branch.
[ ] If applicable, the PR references the bug/issue that it fixes.
[ ] Swagger files are correctly named (e.g. the api-version in the path should match the api-version in the spec).

Quality of Swagger

[ ] I have read the contribution guidelines.
[ ] My spec meets the review criteria:

[ ] The spec conforms to the Swagger 2.0 specification.
[ ] Validation errors from the Linter extension for VS Code have all been fixed for this spec. (Note: for large, previously checked in specs, there will likely be many errors shown. Please contact our team so we can set a timeframe for fixing these errors if your PR is not going to address them).
[ ] The spec follows the patterns described in the Swagger good patterns document unless the service API makes this impossible.
The x-ms-client-flatten extension has been applied in the wrong location.
Unfortunately fixing it would introduce a breaking change so instead we
will just remove the extension.

Hi @jhendrixMSFT, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

It looks like you're working at Microsoft (jhendrix). If you're full-time, we DON'T require a contribution license agreement.

If you are a vendor, DO please sign the electronic contribution license agreement. It will take 2 minutes and there's no faxing! https://cla.azure.com.

TTYL, AZPRBOT;

@amarzavery Here's the PR for the issue we were discussing earlier.

Although this is fixing the application of the client-flattening extension in a correct way, this will cause a breaking change in the DeploymentProperties model class. It will no longer have a property named debugSetting of type "#/definitions/DebugSetting". Instead the class will have a property named "detailLevel" of type "string".
At the CLI/Powershell level the commands to validate deployment and create/update deployment will have to make changes in the command implementation. I dont think there would be any changes in the API of these commands.
The SDK would have breaking changes in the structure of the body parameter (Deployment) being consumed by Deployments_validate and Deployments_CreateOrUpdate operations.
If everyone is ok with this change then feel free to merge the PR.
@jhendrixMSFT, @veronicagg, @lmazuel @yugangw-msft, @jianghaolu, @johnaste, @markcowl

@jhendrixMSFT can you please let us know what is the issue with the current implementation? The current one actually mirrors the service side model - more amenable if we do introduce future properties to debug setting.

@vivsriaus The property of DebugSettings is a primary type so there's nothing to flatten.  Are you saying this is intentional?

@amarzavery PR updated.

 https://github.com/Azure/azure-sdk-for-ruby/pull/645

No modification for NodeJS

 https://github.com/Azure/azure-sdk-for-python/pull/949


:sparkles: Add a way to make sure any input is bytes

Add an ensure_bytes() function that will take either bytes or a pillow image and turn it into bytes. This will make endpoints like create_emoji much easier to implement since we won't have to deal with a pillow import in every file.

I decided this is useless. The user should be forced to use a File object instead.


Integrate GraphQL Birdseye

It would be nice to integrate GraphQL Birdseye (https://birdseye.novvum.io, https://github.com/Novvum/graphql-birdseye) into Altair to represent (visualize) GraphQL API as an interactive graph.
Read more about GraphQL Birdseye here: https://www.novvum.io/post/introducing-graphql-birdseye.
GraphQL Birdseye is available as a React component, but @rohit-ravikoti have separated the pure JS component into its own package called graphql-birdseye-core.
@Novvum, cc.

There's a pending issue that needs to be addressed before proceeding with this.
https://github.com/Novvum/graphql-birdseye/issues/36


cleaned Select from hardcoded Strings

Using hard-coded Strings is not recommended, constant Strings are easier to understand and the code looks cleaner.
Also changed some of the if conditionals to a cleaner look.

we should change the name of method getWhereCond to getWhereClause, since is a getter for the variable whereClause

@sibeliusseraphini is there any chance that someone could check this and other pullrequest pending?

@RoyMontoya I will try to take a look this week

@sibeliusseraphini  ok! thanks!

@RoyMontoya merged thanks, sorry for the delay


Bug: Unable to clean schema that contains a user-defined function with schemabinding in SQL Server

Which version and edition of Flyway are you using?
4.2.0
If this is not the latest version, can you reproduce the issue with the latest one as well?
could not test it, since 5.2.0 is not compatible with SQL Server 2012
Which client are you using? (Command-line, Java API, Maven plugin, Gradle plugin)
Maven plugin, Java API
Which database are you using (type & version)?
SQL Server 2012
Which operating system are you using?
Windows 10
What did you do?
SQL Migration:
IF NOT EXISTS (SELECT name FROM sys.schemas WHERE name = N'MySchema')
  BEGIN
    EXEC sp_executesql N'CREATE SCHEMA [MySchema]';
  END

IF OBJECT_ID('[MySchema].[fn_SomeMyTableValue]') IS NULL -- Check if View Exists
  EXEC('CREATE FUNCTION [MySchema].[fn_SomeMyTableValue] () RETURNS int WITH SCHEMABINDING AS  BEGIN RETURN 1 END') -- Create dummy/empty SP
GO

CREATE TABLE [MySchema].[MyTable] (
  [OrderId] [int] NOT NULL,
  [MyDate] [date] NOT NULL
)

with the following repeatable
CREATE FUNCTION [MySchema].[fn_SomeMyTableValue]
  (
    @OrderId INT
  )
  RETURNS DATE
  WITH SCHEMABINDING
  AS
  BEGIN
    RETURN (
      SELECT MyDate
      FROM [MySchema].[MyTable]
      WHERE OrderId = @OrderId
    )
  END

What did you expect to see?
no error
What did you see instead?
Similar to https://github.com/flyway/flyway/issues/1193, we get a
Caused by: com.microsoft.sqlserver.jdbc.SQLServerException: Cannot DROP TABLE 'MySchema.MyTable' because it is being referenced by object 'fn_SomeMyTableValue'.

after migrating the code and running
            public void migrate(Flyway flyway) {
                flyway.clean();
                flyway.migrate();
            }

Thanks for the report. I can confirm that Flyway clean does indeed not handle the case where a function depends on a table. The opposite works though.


Talos not correct for pybullet - No link should be left without inertia

In pybullet, links should have an inertia and a mass specified even if this is zero.
Otherwise the default mass value is 1 Kg.
See https://github.com/robotology/icub-models/issues/12
A PR follows fixing this issue.

This is awesome!

What I should have mentionned is that I did the corrections for TALOS but not for the other robots...

It looks like that at least the following files would be affected:

kinova.urdf < link name="j2s6s200_end_effector" / >
ur5_joint_limited_robot.urdf:  < link name="tool0" / >
ur3_robot.urdf:  < link name="tool0" / >
ur10_robot.urdf:  < link name="tool0" / >
ur3_joint_limited_robot.urdf:  < link name="tool0" / >
romeo.urdf:  < link name="l_sole" / >
tiago.urdf:  < link name="hand_grasping_frame" / >
tiago_no_hand.urdf:  < link name="base_footprint" / >
romeo_laas_small.urdf:    < link name="AccelerometerDownTorso_sensor" / >
tiago.urdf:  < link name="base_footprint" / >

I performed those fixes for Anymal a while back - but agree we should carry them through all URDFs. I recall at some point some dynamics libraries or simulators did not like "0" inertia either (i.e. crashed) so we went with 1e-6 or 1e-9. I can't recall which where affected though.


Support --route on flutter drive for Web

Use case
I want to run an integration test for flutter web where it opens a unknown route and checks for the 404 page.
flutter drive --target=test_driver/app.dart --no-headless --release --route '/password' lib/routes.dart
Proposal
On the web it seems to ignore the route parameter. On Android it works fine. I think this scenario makes way more sense on the web than on mobile since the user can type any URL in the search bar.

I would like to suggest expanding this issue to browser navigation in general.
I'm attempting to write a battery of driver tests that ensure that if the user enters a given URL after achieving an authenticated state in the app, the correct page is shown. For example: ensuring that a "sign up" page is not served if the user is already signed in. At present, I'm not sure how to write these tests, or if I can.
Ideally, I'd like a Driver API, or something similar that allows me to enter a desired URL at any moment in time as if it were typed into the address bar in the browser.

I'd like to add to this request and make sure that the --route parameter can  be supported in the launch.json file. I'm developing a Flutter Web application and it would be nice to be able to have different configurations that I can select from the dropdown menu (see image below) that would launch with different query parameters in the URL.


snap.getBBox() returns 0 for all attributes

When trying to get the bounding box for the snap element, all values are zero, even if the element.clientWidth returns a non-zero value.
This means I can't do:
  var snap = Snap('#mySVG');
  snap.getBBox().width;

since width returns zero. Instead we must do:
  var snap = Snap('#mySVG');
  snap.node.clientWidth

I would suggest that the snap bounding box takes it values from the clientWidth & clientHeight attributes.
See Fiddle https://jsfiddle.net/ajmas/k2h96c2w/
Tested in Chrome, Safari, Firefox & Opera

getBBox is not the same as clientWidth.

getBBox is not the same as clientWidth.


Publish 12/07/2021, 10:30 AM

Docs Build status updates of commit 7efa6e1:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

exchange/exchange-ps/exchange/Test-Message.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/exchange.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/mapping/serviceMapping.json
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Can't add feed: not found

iOS 17.5.1
Trying to add this: https://www.powerlineblog.com/feed/rdf

The server is returning a 403 Forbidden status code.


fix(examples): hackernews_axum styles href

2023-02-18T15:49:42.957124Z ERROR ❌ error
error from HTML pipeline

Caused by:
    0: error getting canonical path for "/leptos/examples/hackernews_axum/./static/style.css"
    1: No such file or directory (os error 2)

https://github.com/leptos-rs/leptos/issues/537


Add zeroconf/mdns discovery

Discovery is currently stubbed out. Home Assistant is discoverable on _home-assistant._tcp.local.

If it's stubbed out, then why does it say that it's attempting to scan? This is confusing as heck.

I made a preliminary implementation of mdns search. But I still need to structure it a little better before opening a PR

Question about mdns.
When I find a mDNS record on my network, this is what android returns me:
BonjourService{flags=2, domain='local.', regType='_home-assistant._tcp.', serviceName='Home', dnsRecords={base_url=https://RETRACTED.duckdns.org, version=0.102.3, requires_api_password=true}, ifIndex=9, hostname='Home._home-assistant._tcp.local.', port=8123}
<IP_ADDRESS>

The thing is, using my duckdns external URL, I'm remapping the port to 1220, so if I would use that URL with the port that mDNS returns, it wouldn't work.
If I use the discovered IP address, and the port, it will work. But I have no way to know if ssl is required or not...
Should I just assume that http://IPADDRESS:PORT works?

You should update your config to make sure that your base url includes the port that you use.
We should initially try to connect to the base url.
If that won't work, we can do the http://IPADDRESS:PORT but it means we need to disable SSL verification in case the user uses Let's Encrypt (hence why we should use the base url)

If the base URL fails, I can also check if starts with https and use that with the IP address

@marciogranzotto have you made progress on your implementation ?


Implement WPS-T execute method

Implemented and tested via Postman, Jupyter Notebook using a stub Hello World PGE.

Investigate whether each request submits the same job twice.

Namrata found and fixed the problem.


add option to exit out of online_check

Solves #19

@rpotter12 , we don't need the second while loop. The changes I made basically track the status of the user, and asks user after one minute if they are still interested to know the status of user. What we can do is to increase the time gap after which it asks user.
Pressing 'q' to stop the script is almost same as pressing 'ctrl+c' to stop the script. Where is the difference?

@nileshpatra the idea of the script was to track until the user stops the scripts. So there should not be any time limit for tracking.

@rpotter12 if that is the case then the initial code goes well with it - pressing q or ctrl+C, it doesn't make a big difference. If we have to add an option for exit , then we need to change the script for tracking for a finite duration.

@nileshpatra According to the idea the script should not run for a particular duration.


GoDaddy Root Certificate fails - missing CN issue regression

We have an institution that is using the "Go Daddy Class 2 Certification Authority" as the issuer for the certificates in their RADIUS server for the EAP conversation. The raw PEM for this root is here
This root certificate doesn't have a CN/SAN, which is less common, but appears to be enough to trip up the latest geteduroam app. I've also reproduced this on iOS myself.
The app says "Failed to connect - No valid outer EAP type in configuration"
Details of the test environment I reproduced in are below.
iOS: 17.3.1 iPhone 11
geteduroam: Testflight 2.1 (94)
Test Profile : https://cat.eduroam.org/?idp=8196&profile=10643
We have also experienced this before with the older app using the GoDaddy Certificate Chain - G2
The older app applied this pattern which fixed things.
I'm going to ask the institution to add the intermediate to their profile to attempt to work around this as I think this will resolve things...a similar test profile with the intermediate is here and it seems to work, but I can't take it to full completion to be sure as we don't use this root ourselves.

Hard to test by our QA, so directly assigning to Client QA.