[ISSUE] Get entitled attributes API returns an empty response when XACML policy has regex starting with '^'.

Describe the issue:
Get entitled attributes API returns an empty response when a regex with format "^test*" is set for the attribute value in the string-regexp-match function of the XACML policy.
How to reproduce:
add the following XACML policy.
<Policy xmlns="urn:oasis:names:tc:xacml:3.0:core:schema:wd-17"  PolicyId="aaaa_test" RuleCombiningAlgId="urn:oasis:names:tc:xacml:1.0:rule-combining-algorithm:first-applicable" Version="1.0">
   <Target>
      <AnyOf>
         <AllOf>
            <Match MatchId="urn:oasis:names:tc:xacml:1.0:function:string-regexp-match">
               <AttributeValue DataType="http://www.w3.org/2001/XMLSchema#string">^test*</AttributeValue>
               <AttributeDesignator AttributeId="urn:oasis:names:tc:xacml:1.0:resource:resource-id" Category="urn:oasis:names:tc:xacml:3.0:attribute-category:resource" DataType="http://www.w3.org/2001/XMLSchema#string" MustBePresent="true"></AttributeDesignator>
            </Match>
         </AllOf>
      </AnyOf>
   </Target>
   <Rule Effect="Permit" RuleId="PERMIT_RULE"></Rule>
</Policy>

Call the Get entitled attributes API with the following request
curl --request POST \
  --url https://localhost:9443/api/identity/entitlement/decision/entitled-attribs \
  --header 'accept: application/json' \
  --header 'authorization: Basic YWRtaW46YWRtaW4=' \
  --header 'content-type: application/json' \
  --data '{
  "subjectName" : "admin",
  "enableChildSearch" : "false"
}' -k

The following empty response is returned.
{"entitledResultSetDTO":{"entitledAttributesDTOs":[],"advanceResult":false,"message":null,"messageType":null}}%

If the '^' is removed from the "^test*" AttributeValue in the XACML policy the following response is returned.
{"entitledResultSetDTO":{"entitledAttributesDTOs":[{"resourceName":"test*","action":null,"environment":null,"allActions":true,"allResources":false,"attributeDTOs":[]}],"advanceResult":false,"message":null,"messageType":null}}%

Cause:
When the Get entitled attributes API is called the attribute values from the defined policies are extracted and used as the input to evaluate the policies and come up with a result. When a regex like "^test*" is defined in the XACML policy AttributeValue for the match function, this value is taken as the input for evaluating the policy. When "^test*" is used as both regex and input, it will fail to match. Code - https://github.com/wso2/balana/blob/master/modules/balana-core/src/main/java/org/wso2/balana/cond/MatchFunction.java#L415

This issue is being closed due to extended inactivity. Please feel free to reopen it if further attention is needed. Thank you for helping us keep the issue list relevant and focused!


Use containers <IP_ADDRESS>

@arybczak could you elaborate a bit?

@neongreen See https://issues.serokell.io/issue/CSL-1468

stack2nix is unable to change the version of the containers package, so all builds done by nix must be compatible with version 0.5.7, and 0.5.8 isn't available


302 add content for podcasts

Type of change

[x] Content addition
[ ] Bug fix
[ ] Behavior change

Summary of change

Checklist

[ ] The changes follow the contributing guidelines
[ ] I have searched all existing content and verified my additions are novel
and unique
[ ] The content was added to the end of the appropriate data list
[ ] All required content fields are accurately populated
[ ] All items are tagged with all relevant tags

[ ] This fix resolves #
[ ] The changes follow the contributing guidelines
[ ] I have verified the fix works and introduces no further errors

[ ] This change resolves #
[ ] I have confirmed with a maintainer that this change is desired and been
given the go-ahead to work on it in the relevant issue discussion
[ ] The changes follow the contributing guidelines
[ ] I have verified the change and the rest of the site works as expected
[ ] If the change introduces new components, these have been added to
packages/system/src/components with a .stories.tsx file that
adequately renders possible variations of each component.

Going to close this PR and open a new one so we can use the new Netlify previews since the amplify links no longer work.
This will make it easier for reviewers 👍


only the 0,25 tSIT reward did get trough, missing the 1,75 tSIT reward? [BUG]

Not sure if this is a BUG, but reward is only 0,25 tSIT for winning a block.
started the new silicoin miner (v0.09) with a clean install, and succesfully conected to the new blockchain. (Network name: testnet)
Won my first block on this chain a hour ago and won 0,25 tSIT instead of 0,25 + 1,75. Checked my wallet payout adesses and these are set correct in the GUI.
Checked the config file and confirmed the correct walet adress (same as in GUI).

Do you have any poolable plots or are they all OG plots?

have a mix of poolable plots and OG plots atm.
won a second block with no issues so far, (see screenshot) will investigate in to it further tomorrow

So I have a theory, and I might be wrong. I installed the latest version on ubuntu server - cli only. The version is 0.0.8, the node synced, and everything is working BUT I noticed in the cli for silicoin - it has the plotnft command is which has to do with the pooling setup in chia. So I got a feeling one of your poolable plots won a block - but because it was a pool plot, you got 0.25 reward for being the block winner, and 1.75 went to the pool... which in this case either is completely lost since you did not have a pool setup - or its possibly sent to the pool address setup on your chia. So they might have it, or completely discard it since they are only working with chia.
I thought I would combat this by setting up a solo (local) pool, so that if my pooling plots win -I get the 0.25 as a reward, but then the rest of the 1.75 would go to my solo pool wallet. The 2 issues with this right now - GUI doesn't have this functionality, but you might have it over powershell (to create pool). The other issue is, even though I created a solo pool, the contract address is different to that of my chia pool contract adress which is used right not to plot my poolable plots. So I got a feeling once again, that my 1.75 reward for a poolable plot will be sent to that pool (space.pool) and they will disregard it since it isn't chia...
I don't know how farming poolable plots will work with different crypto's like silicoin or flax. Will one have to find a pool that does all of them (or most) - so you dont loose out on rewards....

the issue is across all forks of chia other than the original chia... the OG plots prior to pool will award all of the reward for finding a block/ however the pool plots will only reward 1\8th of the reward and the other 7\8th will not be rewarded because the plots do not specify an address for them to go to when not pool farming

I thought I would combat this by setting up a solo (local) pool, so that if my pooling plots win - I get the 0.25 as a reward, but then the rest of the 1.75 would go to my solo pool wallet. The 2 issues with this right now - GUI doesn't have this functionality, but you might have it over powershell (to create pool).

let me know if your workaround works and how you set it up

So no luck yet.
But at this stage, I fear that farming other coins with poolable plots are just not going to work. Even though I created a new plotnft with silicoin, the contract address needed to create a poolable plot is different from the chia plotnft contract address, so I won't be able to claim the other 1.75 even though it is a solo pool. Eg.
CHIA PLOTNFT:
Wallet height: 583487
Sync status: Synced
Wallet id 2:
Current state: FARMING_TO_POOL
Current state from block height: 551879
Launcher ID: 4571axxxx
Target address (not for plotting): xch19ukfxxxx
Owner public key: aa67dfxxxx
P2 singleton address (pool contract address for plotting): xch1deeuxxxx
Current pool URL: https://eu1.pool.space
Current difficulty: 6
Points balance: 220
Relative lock height: 64 blocks
Payout instructions (pool will pay to this address): xch1y9xxxxx
SILICOIN PLOTNFT:
Wallet height: 25467
Sync status: Synced
Wallet id 2:
Current state: SELF_POOLING
Current state from block height: 13115
Launcher ID: 4800c7xxxx
Target address (not for plotting): tsit1qmxxxx
Owner public key: aa67dfxxxxx
P2 singleton address (pool contract address for plotting): tsit19jxxxx
Claimable balance: 0.0 tsit (0 mojo)
The only variable that is the same in both cases is the owner public key - but that is not used in the plot creation process.
I think flax coin raised a similar issue with chia devs - but couldn't find much clarity in the answers.
https://github.com/Chia-Network/chia-blockchain/issues/7134
I dont know if starting a public pool, that will allow you to farm multiple coins is the answer... or if its even possible.

I have the same problem, I got 0.25 instead of 2, is there any fix to this issue?

same issue here, won 2 block and gotten only 0.25 each, my plots are mostly poolable plots

Apparently the NFT plots wontt work with the software front the fork. At chia they made it so that there isn't a simpele copy and paste solution the
Outlook voor Androidhttps://aka.ms/AAb9ysg downloaden

From: jacklwm90 @.>
Sent: Tuesday, July 20, 2021 3:41:14 AM
To: silicoin-network/silicoin-blockchain @.>
Cc: abspoel99 @.>; Author @.>
Subject: Re: [silicoin-network/silicoin-blockchain] only the 0,25 tSIT reward did get trough, missing the 1,75 tSIT reward? [BUG] (#53)
same issue here, won 2 block and gotten only 0.25 each, my plots are mostly poolable plots
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/silicoin-network/silicoin-blockchain/issues/53#issuecomment-882980077, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AHAJ4J2YS5YYH2KV4K5UNWTTYTH3VANCNFSM5ALISP6A.

same issue here, won 2 block and gotten only 0.25 each, my plots are og and poolable plots mix

Same issue. There is no way that this has to do something with poolable ploots. I'm farming other forks as well, happened only on silicoin.

Upgrade to 0.2.1 and you can claim your missing tSIT


[feature proposal] Multiples Architectures on same model

While issue #110 don't be closed, is there some possibility of Archi allows multiples architecture folders by model? Something like this:

After we implement ArchiMate 3.0 this is the kind of thing we'll be looking at.

Thank you @Phillipus !

Only to help if anybody needs, my workaround for a while was to put a subfolder in Views folder.


Ensure Zambeze Examples are working as part of the CI

Currently, the example image magick files are not working or being run as part of the CI.

There is no way to do this consistently without provisioning all the zambeze agents on the correct VMs and making sure they are up todate.

This PR has hit some roadblocks because to really do this well we need to be using both Ansible and Terraform to provision the infrastructure on the ORC.

The rsync example has been addressed here with @renan-souza help, this pr will be closed. The Globus example will need to be handled separately.


Passing command line arguments to a test

I use node-tap to check "code" quality of Cucumber features that are stored in a Git repository.
Two parameters are necessary to perform this check: the repo location and the branch to check (they vary from one developer to another).
I am looking for the best way to pass those arguments to the test.
I can make it work using something like node test/features.js --repo=... branch=...
Unfortunately the output is a bit messy: I can easily see one test failed but I have to scroll the output for the failed test to see what failed (which is painfull as more than 2000 test are performed).
The spec reporter is great to get a summary of the failed tests but is only available when using the tap command. However this command (tap test/feature.js --reporter=spec) doesn't allow for passing arguments to the test itself.
What approch would you suggest in this situation?

In the very short term, you could use environment variables, since these are passed down to each of the test scripts by default.
As a new feature, I could see some value in something like --test-args="foo --bar=bloo -quUx" or something.  Would that work for you?

Or, actually, scratch that.  Argv is an array, not a string.  I'd follow the pattern of --node-arg and --nyc-arg and have a --test-arg option that you could set multiple times, so for the above example, it'd be something like tap test/*.js --test=arg=foo --test-arg=--bar=bloo --test-arg=-quUx

Okie dokie, landed in 6.2.0.


CSS LS Pass-Through

[x] Completions (doComplete)
[x] Diagnostics (doValidation)
[ ] Highlight Occurrences (findDocumentHighlights)
[ ] Rename Symbol (doRename)
[x] Hover Hints (doHover)
[x] Find References (findReferences)

Not planned for now

findDefinition (is only local, we are better)
findColorSymbols
doCodeActions
findDocumentSymbols (unless needed by doRename)

Closing. Implemented for all our features, and will be activated for new features as developed.


Graphics (background image, logo/favicon)

We need a header/footer graphic. The default is to use the same image for both, but it doesn't have to be that way.
We also need a logo and/or favicon (the tiny icon that is shown for tabs and bookmarks)

Regarding a logo (and small favicon), what do you think about:

which I generated w/ DALL-E.

Not going to lie, that looks like it was generated by an AI and it's not my favorite thing in the world 😄 Maybe something with a similar theme but a little more minimal/logo style. I can play with some image generation tools too and see if I can get something that fits what I'm imagining.

Yeah, fair enough bc that's exactly what it is :D
@huguesrichard also mentioned that we could use it as a template but make a more minimalistic Inkscape version of it. We can also think about smt totally different (maybe even during the retreat...).


⚠ Bofrost.at has degraded performance

In a480282, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1415 ms

Resolved: Bofrost.at performance has improved in 6c101f7.


Fix test clients

Deleted the old test-clients
Added testMode package, where there are two components: adapter and consumer. These components are only enabled if fint.provider.test-mode is enabled. When test-mode is enabled you will be able to test the entire flow through the provider.
Added localprofile. This might not be necessary, do we need both a test and local profile? Perhaps you can see if we can merge these @asgeirn ?

In general I found the issue with the old test-clients was that the code was in a separate project. It was difficult to keep it up-to-date. By moving it into the provider, and making it easy to disable/enable it should hopefully be easier to maintain.

Coverage decreased (-10.8%) to 77.174% when pulling dcf904653cb05a046ec1a8fb9914c5b3580956d8 on fix-test-clients into e765bdd1a7560fbd8b5831e8c774daa445b0b2ad on master.

The loopback test is slow, unfortunately, but does wonders for test coverage!  I can't think of any nice ways to avoid the two sleeps to ensure that things get time to flow across.

OK to merge, @fsjovatsen ?

Why delete the test that verified that the conditional beans were not loaded when using the default profile? (TestModeDisabledSpec or something)
I think it still provides value.

@fsjovatsen @asgeirn this should be merged now I think. Any further comments, or can I close this PR?

Go ahead and close, all good! 👍


[not yet actionable] Add Hangouts Meet video call API

Apparently, Google's deprecation process may eliminate the convenient Hangouts "start a video call" integration we have (one won't be able to just generate a random URL).  It's hard to know from what they've published so far, but https://gsuiteupdates.googleblog.com/2019/01/upcoming-hangouts-service-consolidation.html suggests that it may be removed.  Though since our integration is only for G Suite, it sounds like we have at least until October 2019.
Looking at https://cloud.google.com/blog/products/application-development/hangouts-meet-now-available-in-google, it looks like the successor product (Hangouts Meet) uses the Google Calendar API and supports creating an event (+ video call, if desired) using that system.  See also
https://developers.google.com/calendar/create-events#conferencing.
@skunkmb @hackerkid FYI.  I'm not sure it makes sense to tackle this now because it's possible Google will add the API we want between now and October (or announce they're not actually breaking the API we're using as part of the Hangouts Meet rebrand).

Though since our integration is only for G Suite, it sounds like we have at least until October 2019.

Is this really the case?  I get: "This meeting has been updated to use Hangouts Meet. Contact the organizer for new joining details" when clicking on a video call link from zulip.

It's possible Google changed their schedule.  We'll look into this.

I am also waiting for this issue to integrate teaching Chinese online

Hi!
Does it make sense to remove the "[not yet actionable]" from the title?
For what I understand, it can be done using the calendar API.

It can't be done easily with the calendar API -- you have to create a meeting on some calendar to do it, and so we'd need to have some calendar that gets spammed with hundreds of meetings as folks use the feature.
That API is clearly not intended to offer a "create a call" feature in a third-party app, and I'd worry that we'd run into ToS or other trouble adding a feature of that form.
I suppose what we could do today is make the button take you to meet.google.com/new and at least save a bit of clicking to get a new meeting with a URL?

That would be good enough for now to be honest

We can create a new meeting by simply navigating to http://meet.google.com/new, which automatically creates and redirect to a meeting. We could fetch it and get the redirection URL to share to users.

That wont work after September 30th once the service becomes paid again.

That wont work after September 30th once the service becomes paid again.

https://apps.google.com/intl/en/meet/pricing/

There's a free tier listed as "Always free".

The free tier was up to 24h meetings, going down to 1h meeting maximum. Always free, but reduced :disappointed:

Nice to know, I think they want to kill hangout so they are making meet somewhat free?

How is any of this related to the issue? Even if it there wasn't a free version at all this would still be a valid feature request.

Agreed re: it being a reasonable feature regardless of the free/paid status.  But the problem is that there isn't an API.

We can create a new meeting by simply navigating to http://meet.google.com/new, which automatically creates and redirect to a meeting. We could fetch it and get the redirection URL to share to users.

I don't believe there's a way we can automate this as a feature in Zulip -- I don't think this does anything without a logged in Google account, and we can't manipulate the user's browser to do this and then get back the URL.
Or is the idea that we'd just have the button open https://meet.google.com/new and the user copy-pastes the URL into Zulip manually?  I'm not sure that version is a feature that'd help much (and it'd be inconsistent UI with our other video call integrations, though maybe that's not the end of the world).

I was thinking: Wouldn't be possible to create a private "Zulip-meetings" calendar in the user's Google account and add the events with the Google Meet room in there? If the user wants to remove everything, then they can just remove the whole calendar.
The API for creating calendar events is: https://developers.google.com/calendar/v3/reference/events/insert .
More info for the Request: https://developers.google.com/calendar/v3/reference/events
People have it working here: https://issuetracker.google.com/issues/167260246 (this was a bug report, but includes the solution of how to create events with video-calls). Basically, when sending the event creation add:
{
	"conferenceData": {
		"createRequest": {
			"conferenceSolutionKey": {
				"type": "hangoutsMeet"
			},
			"requestId": "random_unique_string_to_identify_the_meeting"
		},
	},
	"conferenceDataVersion": 1
}

That should allow Zulip to retrieve a Google Meet link and show it as it does with Jitsi.

Seems possible, though sorta ugly.  We'd certainly integrate something of that form if someone were to prototype it.

I can try to implement it but I would need some help. For now I have these questions:

Where should the Google API calls be done? Front or backend?
How the login into google should be done? Opening a browser tab or in the app itself?
Do you have a Google App already that can be used?

hey @timabbott , are you still using google meet url generation in your app ?

Where should the Google API calls be done? Front or backend?

It depends on the authentication, but it might need to be on the backend if there's a Zulip-side secret.  If Google's authentication APIs allow it, one could imagine a frontend-only implementation, but I think that's unlikely to be supported.

How the login into google should be done? Opening a browser tab or in the app itself?

An Oauth flow like we use for Zoom (and soon Webex) is probably best (https://github.com/zulip/zulip/pull/16633).

Do you have a Google App already that can be used?

There is one for Google authentication; I'm not sure if one could reuse those credentials for this or we'd need something separate.
https://zulip.readthedocs.io/en/latest/production/authentication-methods.html#plug-and-play-sso-google-github-gitlab

I'm still struggling with creating a meet link,
Event is getting created but im not getting the meet link
This is my current request
event = {
      "calendarId": "primary",
      "end": {
        'dateTime': '2020-12-28T17:00:00-07:00',
        'timeZone': 'Asia/Kolkata'
      },
      "start": {
        'dateTime': '2020-12-28T09:00:00-07:00',
        'timeZone': 'Asia/Kolkata'
      },
      "conferenceData": {
		  "createRequest": {
			  "conferenceSolutionKey": {
				  "type": "hangoutsMeet"
			    },
			  "requestId": "RandomStringBrr"
		    },
	    },
      "conferenceDataVersion": 1,
      "summary": "titles are cool"
      }

Any news/progress regarding Google Meets integration? That's a much needed features :-)

Agreed re: it being a reasonable feature regardless of the free/paid status. But the problem is that there isn't an API.

We can create a new meeting by simply navigating to http://meet.google.com/new, which automatically creates and redirect to a meeting. We could fetch it and get the redirection URL to share to users.

I don't believe there's a way we can automate this as a feature in Zulip -- I don't think this does anything without a logged in Google account, and we can't manipulate the user's browser to do this and then get back the URL.
Or is the idea that we'd just have the button open https://meet.google.com/new and the user copy-pastes the URL into Zulip manually? I'm not sure that version is a feature that'd help much (and it'd be inconsistent UI with our other video call integrations, though maybe that's not the end of the world).

I don't know if it's new, but there actually is a way to create a new meeting. If you go to http://g.co/meet/MEETING_NAME, anyone in your organization will be redirected to the same meet upon entering.
If we combined that with a unique identifier (time + stream/topic/DM participants should be good), we could create a /meet command which will create a new meet link and send it in the channel.

I get

on going http://g.co/meet/alsdkfjaasdfbab13523

Interesting, the link works when I try it.
Maybe this only works if you are currently logged into a google account which is part of an organization? I seem to get the same error when incognito or logged into another (regular) account. :disappointed:

Yeah, works for my zulip.com email account too.

Zapier Google Meet integration could be a good solution to this if anyone wants to try that out - https://zapier.com/apps/google-meet/integrations/zulip

Our organization would really appreciate google meet video integration. Is there a way to impact the product roadmap of zulip through premium support channels or similar?

@aleksanb Yes! Please reach out to<EMAIL_ADDRESS>to discuss details.

I don't know if it's new, but there actually is a way to create a new meeting. If you go to http://g.co/meet/MEETING_NAME, anyone in your organization will be redirected to the same meet upon entering.

Thank you for posting this - knowing this exists gets us 90% of the way to solving the need for us!
We can add a linkifier for this, but (unless I'm mistaken) there's no way to get the resulting link to include some random characters?
E.g. We'd do something like G:test expanding to https://g.co/meet/test but there's a risk that someone else has the genius idea to also do G:test and would unexpectedly end up in an existing meeting instead of forming a new/unique one
The linkifier approach (even if there's a solution to the unique naming aspect) also lacks various niceties vs. what I'd want from a full integration:

annotate the link with an emoji (e.g. a Google Meet icon) to make it clearer what clicking it gets you to (that would be equally applicable to other custom linkifiers too!)
automatically switch the user status to "In a meeting" for anyone who clicks the link
pull the stream topic into the meeting name/link (e.g. if I post /meet in a stream with topic "Google Meet idea" it should post something like: 📆 Google Meet idea)

though my particular example is not a valid meeting name. I can't find the Google docs about this feature to confirm what the rules are, but it highlights that simply copying the Zulip stream topic to Google Meet meeting name would not be so simple + obviously this doesn't address how /meet should behave in a PM context
this also gets very close to #22649

I was trying to use Zapier integration to check if its possible to get a meeting link out of Google somehow.
So if we create a bot via webhook (in my case sending a message was a trigger):

Zapier is able to trigger Google Meet:

but actually it is Google Calendar integration.
I think conceptually is ok . If I'm as a user want Zulip to create such links I would need to allow Zulip to create and manage events in my google calendar. So everytime I want to send a meeting link to somebody there will be event in my calendar, for 5-15 mins meeting starting at the time of such link creation in Zulip.
A person I'm inviting will be among the people who is invited to this event (if it is possible to avoid that we could try to not send such an invite, but its not a bad thing, sometimes people want to join a meeting from the phone and they don't have the Zulip mobile app but they can access email)
There are different kinds of meeting links possible
https://developers.google.com/calendar/api/guides/create-events#conferencing
I don't know which one Zapier uses, but they have an old key name for this URL from google:
 and it says hangouts meet. And there is other data in the returned object:

An additional advantage of making such integration is the possibility of being in Google Marketplace, which even Zoom does:
https://workspace.google.com/marketplace/category/work-from-everywhere

Should the issue title be renamed to say "Google Meet" instead of "Hangouts Meet"? I wasn't able to find this issue directly. Had to search for "Hangout" first.

+1 for Google met integration (with option for user to choose between Jitsi and Google). Selfishly we have a paid Google subscription so using that along with the superior Google Meet would work for us.
In terms of user experience, the current meeting invocation method with Jitsi works well.
Incidentally, are there plans to add text triggered shortcuts (e.g. / or G:) to allow video chat or other functionality (e.g. voice call, attachment, etc) to Zulip?

Incidentally, are there plans to add text triggered shortcuts (e.g. / or G:) to allow video chat or other functionality (e.g. voice call, attachment, etc) to Zulip?

I presume you saw my earlier comment about the linkifier workaround?
Referring back to that same comment, this is the biggest miss for me:

automatically switch the user status to "In a meeting" for anyone who clicks the link

The rest of the stuff I mentioned "would be nice", but having Zulip automatically flip status when you click a meeting link would be much appreciated by my already overworked mouse.
A way to designate a linkifier as "the meeting link" would make this fully flexible for any meeting integration (not just those of the Google flavour).

Hi @dransome. Thanks for the suggestion.
I just tested the linkifier solution. But GMeet has wants meeting URL format to be https://meet.google.com/xxx-yyyy-zzz. I don't think a linkifier would work here as the meeting ID has to be in a specific format. Users can't just type what they want.
Try this URL: g.co/meet/greatuniquemeeting

@johnathanz

With linkifier, user would type something likegreatuniquemeeting and following link would be generated, which won't work.

For us, that's exactly what happens.
We have this linkifier set:
G:(?P<meet>[a-zA-Z0-9_-]+) https://g.co/meet/{meet}

And on access it redirects to something like https://meet.google.com/izf-rizt-dio?authuser=0 (this was the actual URL I landed at from following your link).
I believe that Google call this feature "creating meetings with nicknames" and it's only available to Google Workspace users. (see Google Workspace Help > Learn about meeting codes in Google Meet > Meetings with Google Workspace nicknames)
There is a Google Meet REST API where a new meeting (meeting space) can be created via API. Authentication is via OAuth 2.0.

great thanks for the explanation @dransome & the linkifier regex. This now works for us! 🙏


requests About headers message sorting

session.py
def merge_setting(request_setting, session_setting, dict_class=OrderedDict):
    """Determines appropriate setting for a given request, taking into account
    the explicit setting on that request, and the setting in the session. If a
    setting is a dictionary, they will be merged together using `dict_class`
    """

    if session_setting is None:
        return request_setting

    if request_setting is None:
        return session_setting

    # Bypass if not a dictionary (e.g. verify)
    if not (
            isinstance(session_setting, Mapping) and
            isinstance(request_setting, Mapping)
    ):
        return request_setting

    merged_setting = dict_class(to_key_val_list(session_setting))
    merged_setting.update(to_key_val_list(request_setting))

    # Remove keys that are set to None. Extract keys first to avoid altering
    # the dictionary during iteration.
    none_keys = [k for (k, v) in merged_setting.items() if v is None]
    for key in none_keys:
        del merged_setting[key]

    return merged_setting

Expected Result
Host: www.example.com
Referer: http://httpbin.org/get
User-Agent:Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/73.0.3683.103 Safari/537.36
Accept-Encoding: gzip, deflate
Accept: /
Connection: keep-alive
`
Actual Result
User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/73.0.3683.103 Safari/537.36
Accept-Encoding: gzip, deflate
Accept: /
Host: www.example.com
Referer: http://httpbin.org/get
Connection: keep-alive
Reproduction Steps
    import requests

    headers = {
        'Host': 'www.example.com',
        'Referer': 'http://httpbin.org/get',
        'User-Agent':'Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/73.0.3683.103 Safari/537.36',
        'Accept-Encoding': 'gzip, deflate',
        'Accept': '/'
    }
    proxies = {
        'http': 'http://<IP_ADDRESS>:8888',
        'https': 'https://<IP_ADDRESS>:8888'
    }

System Information
$ python -m requests.help

{
  "chardet": {
    "version": "3.0.4"
  },
  "cryptography": {
    "version": "2.4.2"
  },
  "idna": {
    "version": "2.8"
  },
  "implementation": {
    "name": "CPython",
    "version": "3.7.1"
  },
  "platform": {
    "release": "10",
    "system": "Windows"
  },
  "pyOpenSSL": {
    "openssl_version": "101000af",
    "version": "18.0.0"
  },
  "requests": {
    "version": "2.22.0"
  },
  "system_ssl": {
    "version": "1010009f"
  },
  "urllib3": {
    "version": "1.24.1"
  },
  "using_pyopenssl": true
}

This command is only available on Requests v2.16.4 and greater. Otherwise,
please provide some basic information about your system (Python version,
operating system, &c).

@HC-miss most of us are aware of this "bug" use OrderedDict from collections to get around it.


remove getfsstat wrapper, the original issue has since been resolved

The original issue appears to have been: https://github.com/golang/go/issues/6588 , but that was resolved in Go 1.3.
I've verified the workaround is no longer required on go1.6 darwin/amd64 and in the upcoming OpenBSD backend on go1.5.3 openbsd/amd64

LGTM


createAddress should take an opts param

The server-side createAddress accepts an opts param, which optionally takes an ignoreMaxGap option. Currently, the client-side createAddress doesn't accept the opts param so it's not possible to pass that option.

We agree on exposing the parameter in BWC, but it is not a priority right now. If you want to move forward an implement it we can review the code and merge it.

Thank you, here it is: https://github.com/bitpay/bitcore-wallet-client/pull/197


fix(elixir): handle last block bug for @class.inner

With the previous matches if you triggered a @class.inner change and you happened to be in the last block of a module it would only change that block instead of deleting all the blocks of the module.
For example with a file such as this where | is the cursor:
defmodule Test do
  def hello() do
    IO.puts("hi")
  end

  def oi() do
    |IO.puts("oi")
  end
end

After the edit you'd have:
defmodule Test do
  def hello() do
    IO.puts("hi")
  end

  |
end

The previous approach was using a single query and then trying to build
a range over all children of a block, or in the case of a single child
using it as the start and end. I could not for the life of me figure out
a way to write this to work. As soon as I broke it out to two queries:
one with a make range and one that just captures the one child things
worked.

Thank you. With this improved version however, the same problem exists when you try to delete in the first hello() function.

Ah your are right. More work to do on this. I will pull back to draft.

Changing my approach on this. Before my initail PR I had hacked on this for a few hours. What I have here consistenly works but it is a little annoying to have to define things as two separate queries. It feels like there is a way to make it work for both cases in a single query. But I for the life of me couldn't figure out how to make #make-range fork for both situations.

Thank you!


Overhaul Node3D documentation

Closes https://github.com/godotengine/godot/issues/87896
Related to several other past efforts, but especially the Basis and Transform3D ones.
This PR aims to completely overhaul Node3D's class reference in an attempt to make it more easy to digest, as well as bring on par with the rest of the documentation.
Why? It generally suffers from the same issues highlighted in the past. The most notable of which being that a lot of descriptions are dull, succinct, or put the focus on the wrong subject:

"Set subgizmo selection for this node in the editor."
"Updates all the [Node3D] gizmos attached to this node."
"Transforms [param global_point] from world space to this node's local space."
"Rotates the local transformation around the X axis by angle in radians."

More specific notes:

Remove any leftover references to "Spatial" nodes from the class reference.
Modified to_local and to_global's descriptions to imply they do return something.
Elaborate on when visibility_changed is emitted.

The sentence is almost word-for-word from the way some NOTIFICATION constants are written in the Node class.

Revamp the descriptions of NOTIFICATION_TRANSFORM_CHANGED & similar.

"In order for [constant NOTIFICATION_TRANSFORM_CHANGED] to work, users first need to ask for it, with [method set_notify_local_transform]."
a grand majority of Node3Ds will receive NOTIFICATION_TRANSFORM_CHANGED without needing for the "user" to opt-in. They need this otherwise they would not work.

This PR may contain commented XML. I use it as a point of reference or to symbolize alternatives of the same sentence. Even if this is a draft, feedback on things worth mentioning is still very welcome.

Node2D has a lot of the same properties as the 3D version, and often with very short descriptions in the docs. So if you add improved descriptions of the concepts in the Node3D versions perhaps some of them can be applied to Node2D as well? Like for example what the global_position and global_transform properties do and how they work.

That's the idea! One thing at the time. I decided to tackle Node3D first because it's fresh on the mind after both https://github.com/godotengine/godot/pull/87175 and https://github.com/godotengine/godot/pull/87334.

I have updated the PR with just... more. I am still not fully happy on what is here, as there's a constant fear of losing important information through all of this. So this is still gonna be WIP for a bit more time until I come back with a fresher mind.

Marking this as ready to be lynched by anyone more savvy than me, no beating around the bush.

Given your recent review I'll revisit. The "local" distinction is bit overwhelming to think about, but I must pursue for the sake of a nicer class reference.

I have rebased. My general confusion stems from the original documentation's definition of "local" which has been irritatingly misleading.

BTW I often use local/global space etc. but I think it might be better if in the docs we'd refer to them explicitly as "local/global coordinate system", it should be clearer and more precise at the same time.

In general, Godot's properties are parent-relative by default unless further qualified, where the word global qualifies it as fully global relative to the world space, and the word local qualifies it as relative to within the node's own transform. For the docs where we need to clarify this, let's use "parent-relative", not "local".

Thank you @kleonc and @aaronfranke for writing detailed explanations on the distinction between these 3 "spaces"/"coordinate systems", but we have to discuss this. We need a consistent way to refer to these in order to reduce ambiguity as much as possible.
It doesn't help that in some other places, different terms are used to refer to the same concept ^1. I myself understand now that both "object-local" and "parent-local" spaces are, in fact, "local", but a problem arises when both are summarised with that single word.
Right now, in this PR, I am using the term "parent space". This term is never mentioned anywhere in the class reference. Even though a lot of wording can still be improved, this has been done for sake of accuracy first.

Pushed an update to address all of the above. But there's just a bit more to do.

I did not go with the shortened version of each suggestion, as I feel like the descriptions can afford to be explicit here.
I am not sure it's necessary to state that, for example, rotate_x is equivalent to using an axis of Vector3(1, 0, 0 in rotate. I am not dying on that hill, though.
I like saying "basis" instead of "local transformation". That's actually genius. The local transformation's origin is always void, so this simplifies things a lot to me.

I think mentioning that the position is unaffected could still be warranted, though. Just to make it clearer.
Perhaps in global_basis and basis's own description these are the object-local transformations?

I have handled similar descriptions as suggested, but the idea of saying "around the global [param axis] going through this node's origin" interests me. It sounds a whole lot easier to imagine the whole operation that way.
I have written a note encouraging to use translate_object_local instead of translate

I think mentioning that the position is unaffected could still be warranted, though. Just to make it clearer.

Nothing against making things clearer.

Perhaps in global_basis and basis's own description these are the object-local transformations?

If by "object-local transformation" you mean non-translating / keeping the position of the given object/node unchanged then sure you could say so. But I wouldn't add anything like that, this will be confusing. There's already Node3D::translate_object_local method. If "object-local" would mean "not changing position" then how could you ever "translate-object-local"? Makes no sense. Let's not add potential additional confusion.
The "object-local" being used means the same as what we already call just "local" in the context of the given node/object. Like:

parent == parent-local == (parent-object)-local
local == (self/this)-local == (self/this-object)-local == object-local

In fact I was wondering whether we should maybe use e.g. parent-local/self-local instead of just parent/local. Local is always relative to something, so it's potentially easy to misinterpret. Object-local is meant to be more clear, as it specifies that the "local" refers to the given object, not its parent etc. But I think that in context of e.g. Node3D it's already clear that "local" refers to this specific node/object? :thinking:
Anyway, if we'd want to use e.g. "object-local" instead of just "local" then we should probably do so consequently across the whole docs. AFAICT currently we use just "local" to refer to the given object/node, so I'm saying let's keep using it.

I have handled similar descriptions as suggested, but the idea of saying "around the global [param axis] going through this node's origin" interests me. It sounds a whole lot easier to imagine the whole operation that way.

Again, nothing against clarifications. The purpose of the docs is not to be as concise as possible (while being correct). So if the descriptions can be made a little longer but easier to grasp for "a regular user" then sure it's welcomed (we'll try to keep them correct).

I have written a note encouraging to use translate_object_local instead of translate

If we want to encourage using translate_object_local then maybe we should make translate's description only redirect to translate_object_local? Something like: "Same as [method translate_object_local], for compatibility reasons. Prefer using it instead." This way the user would have to check it out and hence more likely use it? :thinking: Just throwing an idea, I don't have a strong opinion on that, it's fine as is / up to you.
But might be a good to add additional note/warning to translate like "(For compatibility reasons, )this operation is not calculated in the parent space. To translate in parent space use node_3d.position += offset.".

I have updated to add "and preserves the position" where appropriate, as well as update translate and translate_object_local again.

Anyway, if we'd want to use e.g. "object-local" instead of just "local" then we should probably do so consequently across the whole docs. AFAICT currently we use just "local" to refer to the given object/node, so I'm saying let's keep using it.

Mhm. Hence my reluctance to say object-local space. If it needs to be clarified it needs to be done in one huge swoop across the docs.

I have handled similar descriptions as suggested, but the idea of saying "around the global [param axis] going through this node's origin" interests me. It sounds a whole lot easier to imagine the whole operation that way.

Again, nothing against clarifications. The purpose of the docs is not to be as concise as possible (while being correct). So if the descriptions can be made a little longer but easier to grasp for "a regular user" then sure it's welcomed (we'll try to keep them correct).

Problem here is that I'd have to give up on directly mentioning the basis otherwise the sentence doesn't make sense, and for consistency that may need to escalate for the surrounding descriptions as well.

Same as [method translate_object_local], for compatibility reasons. Prefer using it instead.

This sounds exactly like a deprecation message, but the method is not deprecated until we decide that separately. So it's a bit odd to do this.

I have updated to add "and preserves the position" where appropriate, as well as update translate and translate_object_local again.

Anyway, if we'd want to use e.g. "object-local" instead of just "local" then we should probably do so consequently across the whole docs. AFAICT currently we use just "local" to refer to the given object/node, so I'm saying let's keep using it.

Mhm. Hence my reluctance to say object-local space. If it needs to be clarified it needs to be done in one huge swoop across the docs.

I have handled similar descriptions as suggested, but the idea of saying "around the global [param axis] going through this node's origin" interests me. It sounds a whole lot easier to imagine the whole operation that way.

Again, nothing against clarifications. The purpose of the docs is not to be as concise as possible (while being correct). So if the descriptions can be made a little longer but easier to grasp for "a regular user" then sure it's welcomed (we'll try to keep them correct).

Problem here is that I'd have to give up on directly mentioning the basis otherwise the sentence doesn't make sense, and for consistency that may need to escalate for the surrounding descriptions as well.

Same as [method translate_object_local], for compatibility reasons. Prefer using it instead.

This sounds exactly like a deprecation message, but the method is not deprecated until we decide that separately. So it's a bit odd to do this.


Dropdown now escape the space character

Dropdown now escape the space character of a collection.
fix #285

Codecov Report

Merging #288 into 2.x will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##              2.x     #288   +/-   ##
=======================================
  Coverage   76.67%   76.67%
=======================================
  Files          39       39
  Lines         763      763
=======================================
  Hits          585      585
  Misses        178      178

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update e80fe25...40c436d. Read the comment docs.


update glide.yaml

Signed-off-by: zhouhao<EMAIL_ADDRESS>

ping @coolljt0725 @xiekeyang @stevvooe

If updating are fixing some problem?

Not currently, just an updated version.

LGTM

LGTM


⚠ gorzdrav.org has degraded performance

In 0977294, gorzdrav.org (https://gorzdrav.org) experienced degraded performance:

HTTP code: 200
Response time: 4427 ms

Resolved: gorzdrav.org performance has improved in be80de7.


Rewriting HeaderString to use absl::variant. See #9593

Signed-off-by: Adi Suissa-Peleg<EMAIL_ADDRESS>Description: Rewriting HeaderString to use absl::variant instead of union. Removed Type::Dynamic, updated the implementation, and tests.
Risk Level: Low/Medium - being used by the header_map, et al.
Testing: Kept and updated most unit tests as before. Removed one unit test that accessed a region of memory that was out of scope.
Docs Changes:
Release Notes:
Fixes #9593

Also ran the perf tests for this PR (test/common/http:header_map_impl_speed_test).
There's some performance hit, and further investigation might be required.

Thanks for working on this. Haven't reviewed yet, but just wanted to say that I'm not surprised there is some perf delta, though it would be worth it to understand where the delta is maybe by taking a look at a cachegrind run? IMO some perf delta is probably worth it given the code simplification but very happy to hear what others think on this.

I'll look into the detailed results of cachegrind and submit the fixed PR tomorrow morning.

After removing some unnecessary things, fixing some small issues, and running the performance tests (test/common/http:header_map_impl_speed_test), we've got the following results (new version called "w/ absl::variant 2"):

Seems that most of the functions are a bit faster than the original implementation.
There's some penalty for the SetInteger tests and the HeaderMapImplPopulate test.

Sorry do you mind merging master and I will take another pass?
/wait

Let me know when you are done addressing @asraa comments and I will take a look. Thank you!
/wait

@mattklein123 I think it would be better to change the call to remove to_lower_table and call ascii_tolower instead, in a different PR, as it has implications on other parts of the code.

Sounds good I will review.

Fuzz approved, thanks for adding coverage for both inline/reference header fields and inlineTransform.

Looks like a side effect of this PR is the building on RHEL 7 now breaks. Error is below building on RHEL 7 with gcc 8 and bazel build -c opt --copt=-DENVOY_IGNORE_GLIBCXX_USE_CXX11_ABI_ERROR=1 //source/exe:envoy-static.stripped. Any workarounds possible? /cc @asraa
ERROR: /home/<USERNAME>/Library/envoyproxy/envoy/source/common/http/BUILD:256:1: C++ compilation of rule '//source/common/http:header_map_lib' failed (Exit 1) gcc failed: error executing command /opt/rh/devtoolset-8/root/usr/bin/gcc -U_FORTIFY_SOURCE -fstack-protector -Wall -Wunused-but-set-parameter -Wno-free-nonheap-object -fno-omit-frame-pointer -g0 -O2 '-D_FORTIFY_SOURCE=1' -DNDEBUG ... (remaining 86 argument(s) skipped)

Use --sandbox_debug to see verbose messages from the sandbox
In file included from external/com_google_absl/absl/container/inlined_vector.h:53,
                 from bazel-out/k8-opt/bin/include/envoy/http/_virtual_includes/header_map_interface/envoy/http/header_map.h:18,
                 from bazel-out/k8-opt/bin/source/common/http/_virtual_includes/header_map_lib/common/http/header_map_impl.h:9,
                 from source/common/http/header_map_impl.cc:1:
external/com_google_absl/absl/container/internal/inlined_vector.h: In constructor 'Envoy::Http::HeaderString::HeaderString()':
external/com_google_absl/absl/container/internal/inlined_vector.h:437:5: error: '<anonymous>.absl::inlined_vector_internal::Storage<char, 128, std::allocator<char> >::data_' is used uninitialized in this function [-Werror=uninitialized]
     data_ = other_storage.data_;
     ^~~~~
external/com_google_absl/absl/container/internal/inlined_vector.h: In member function 'void Envoy::Http::HeaderString::setInteger(uint64_t)':
external/com_google_absl/absl/container/internal/inlined_vector.h:437:5: error: '<anonymous>.absl::inlined_vector_internal::Storage<char, 128, std::allocator<char> >::data_' may be used uninitialized in this function [-Werror=maybe-uninitialized]
     data_ = other_storage.data_;
     ^~~~~
external/com_google_absl/absl/container/internal/inlined_vector.h:437:5: error: '<anonymous>.absl::inlined_vector_internal::Storage<char, 128, std::allocator<char> >::data_' may be used uninitialized in this function [-Werror=maybe-uninitialized]
     data_ = other_storage.data_;
     ^~~~~
external/com_google_absl/absl/container/internal/inlined_vector.h:437:5: error: '<anonymous>.absl::inlined_vector_internal::Storage<char, 128, std::allocator<char> >::data_' may be used uninitialized in this function [-Werror=maybe-uninitialized]
     data_ = other_storage.data_;
     ^~~~~
external/com_google_absl/absl/container/internal/inlined_vector.h: In member function 'void Envoy::Http::HeaderString::setCopy(const char*, uint32_t)':
external/com_google_absl/absl/container/internal/inlined_vector.h:437:5: error: '<anonymous>.absl::inlined_vector_internal::Storage<char, 128, std::allocator<char> >::data_' may be used uninitialized in this function [-Werror=maybe-uninitialized]
     data_ = other_storage.data_;
     ^~~~~
external/com_google_absl/absl/container/internal/inlined_vector.h:437:5: error: '<anonymous>.absl::inlined_vector_internal::Storage<char, 128, std::allocator<char> >::data_' may be used uninitialized in this function [-Werror=maybe-uninitialized]
     data_ = other_storage.data_;
     ^~~~~
external/com_google_absl/absl/container/internal/inlined_vector.h: In member function 'void Envoy::Http::HeaderString::append(const char*, uint32_t)':
external/com_google_absl/absl/container/internal/inlined_vector.h:437:5: error: '<anonymous>.absl::inlined_vector_internal::Storage<char, 128, std::allocator<char> >::data_' may be used uninitialized in this function [-Werror=maybe-uninitialized]
     data_ = other_storage.data_;
     ^~~~~
cc1plus: all warnings being treated as errors
Target //source/exe:envoy-static.stripped failed to build

Thanks.
Interesting, it seems that the field might not be initialized in these
settings.
I'll try to see if the error can be reproduced or if the member can be
explicitly initialized.
On Tue, Feb 18, 2020, 04:22 moderation<EMAIL_ADDRESS>wrote:

Looks like a side effect of this PR is the building on RHEL 7 now breaks.
Error is below building on RHEL 7 with gcc 8 and bazel build -c opt
--copt=-DENVOY_IGNORE_GLIBCXX_USE_CXX11_ABI_ERROR=1
//source/exe:envoy-static.stripped. Any workarounds possible? /cc @asraa
https://github.com/asraa
ERROR: /home//Library/envoyproxy/envoy/source/common/http/BUILD:256:1: C++ compilation of rule '//source/common/http:header_map_lib' failed (Exit 1) gcc failed: error executing command /opt/rh/devtoolset-8/root/usr/bin/gcc -U_FORTIFY_SOURCE -fstack-protector -Wall -Wunused-but-set-parameter -Wno-free-nonheap-object -fno-omit-frame-pointer -g0 -O2 '-D_FORTIFY_SOURCE=1' -DNDEBUG ... (remaining 86 argument(s) skipped)
Use --sandbox_debug to see verbose messages from the sandbox
In file included from external/com_google_absl/absl/container/inlined_vector.h:53,
from bazel-out/k8-opt/bin/include/envoy/http/virtual_includes/header_map_interface/envoy/http/header_map.h:18,
from bazel-out/k8-opt/bin/source/common/http/virtual_includes/header_map_lib/common/http/header_map_impl.h:9,
from source/common/http/header_map_impl.cc:1:
external/com_google_absl/absl/container/internal/inlined_vector.h: In constructor 'Envoy::Http::HeaderString::HeaderString()':
external/com_google_absl/absl/container/internal/inlined_vector.h:437:5: error: '.absl::inlined_vector_internal::Storage<char, 128, std::allocator >::data' is used uninitialized in this function [-Werror=uninitialized]
data = other_storage.data_;
^~~~~
external/com_google_absl/absl/container/internal/inlined_vector.h: In member function 'void Envoy::Http::HeaderString::setInteger(uint64_t)':
external/com_google_absl/absl/container/internal/inlined_vector.h:437:5: error: '.absl::inlined_vector_internal::Storage<char, 128, std::allocator >::data_' may be used uninitialized in this function [-Werror=maybe-uninitialized]
data_ = other_storage.data_;
^~~~~
external/com_google_absl/absl/container/internal/inlined_vector.h:437:5: error: '.absl::inlined_vector_internal::Storage<char, 128, std::allocator >::data_' may be used uninitialized in this function [-Werror=maybe-uninitialized]
data_ = other_storage.data_;
^~~~~
external/com_google_absl/absl/container/internal/inlined_vector.h:437:5: error: '.absl::inlined_vector_internal::Storage<char, 128, std::allocator >::data_' may be used uninitialized in this function [-Werror=maybe-uninitialized]
data_ = other_storage.data_;
^~~~~
external/com_google_absl/absl/container/internal/inlined_vector.h: In member function 'void Envoy::Http::HeaderString::setCopy(const char*, uint32_t)':
external/com_google_absl/absl/container/internal/inlined_vector.h:437:5: error: '.absl::inlined_vector_internal::Storage<char, 128, std::allocator >::data_' may be used uninitialized in this function [-Werror=maybe-uninitialized]
data_ = other_storage.data_;
^~~~~
external/com_google_absl/absl/container/internal/inlined_vector.h:437:5: error: '.absl::inlined_vector_internal::Storage<char, 128, std::allocator >::data_' may be used uninitialized in this function [-Werror=maybe-uninitialized]
data_ = other_storage.data_;
^~~~~
external/com_google_absl/absl/container/internal/inlined_vector.h: In member function 'void Envoy::Http::HeaderString::append(const char*, uint32_t)':
external/com_google_absl/absl/container/internal/inlined_vector.h:437:5: error: '.absl::inlined_vector_internal::Storage<char, 128, std::allocator >::data_' may be used uninitialized in this function [-Werror=maybe-uninitialized]
data_ = other_storage.data_;
^~~~~
cc1plus: all warnings being treated as errors
Target //source/exe:envoy-static.stripped failed to build
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/envoyproxy/envoy/pull/9952?email_source=notifications&email_token=ABEQZZZCRXCTQN5NPXCC3V3RDOSFHA5CNFSM4KRETFU2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEMBGL6Y#issuecomment-587359739,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABEQZZYVSMI4GTUSR5O4NWDRDOSFHANCNFSM4KRETFUQ
.

Looks like a side effect of this PR is the building on RHEL 7 now breaks. Error is below building on RHEL 7 with gcc 8 and bazel build -c opt --copt=-DENVOY_IGNORE_GLIBCXX_USE_CXX11_ABI_ERROR=1 //source/exe:envoy-static.stripped. Any workarounds possible? /cc @asraa

Do you get the same error running external/com_google_absl/absl/container/inlined_vector_test.cc?

@moderation I've attempted to compile it with GCC, but couldn't reproduce the problem.
The errors seem to be coming from abseil. If you can provide me with a docker image to test this on, I may be able to help with this.

@adisuissa I think the issue is specific to RHEL 7 (and possibly Centos) due to the ancient glibc. I suspect using GCC 8 on something like Ubuntu will trigger the issue.
@asraa can you give me a hint on how to run that specific test? What is the Bazel label that I would use as outlined at https://github.com/envoyproxy/envoy/tree/master/bazel#testing-envoy-with-bazel ?

It  might be hackish, but I cd'd in to my bazel cache, in to the external/com_google_absl directory, I copied the WORKSPACE from https://github.com/abseil/abseil-cpp/blob/master/WORKSPACE because I guess pulling external deps writes over their WORKSPACE,  and ran bazel test absl/container:inlined_vector_test and all was well

That worked @asraa ! I confirmed the test passed on my Xubuntu 19.10 with Clang 9. Output from work RHEL 7 is below:
INFO: Analyzed target //absl/container:inlined_vector_test (31 packages loaded, 478 targets configured).
INFO: Found 1 test target...
ERROR: /home/<USERNAME>/.cache/bazel/_bazel_<USERNAME>@<DOMAIN>/3cccb42b5a473b17439089ce85d34673/external/com_google_absl/absl/container/BUILD.bazel:147:1: Linking of rule '//absl/container:inlined_vector_test' failed (Exit 1) gcc failed: error executing command /opt/rh/devtoolset-8
/root/usr/bin/gcc @bazel-out/k8-fastbuild/bin/absl/container/inlined_vector_test-2.params

Use --sandbox_debug to see verbose messages from the sandbox
bazel-out/k8-fastbuild/bin/_solib_k8/libabsl_Sbase_Slibthrow_Udelegate.so: error: undefined reference to 'std::logic_error::logic_error(char const*)'
bazel-out/k8-fastbuild/bin/_solib_k8/libabsl_Sbase_Slibthrow_Udelegate.so: error: undefined reference to 'std::invalid_argument::invalid_argument(char const*)'
bazel-out/k8-fastbuild/bin/_solib_k8/libabsl_Sbase_Slibthrow_Udelegate.so: error: undefined reference to 'std::domain_error::domain_error(char const*)'
bazel-out/k8-fastbuild/bin/_solib_k8/libabsl_Sbase_Slibthrow_Udelegate.so: error: undefined reference to 'std::length_error::length_error(char const*)'
bazel-out/k8-fastbuild/bin/_solib_k8/libabsl_Sbase_Slibthrow_Udelegate.so: error: undefined reference to 'std::out_of_range::out_of_range(char const*)'
bazel-out/k8-fastbuild/bin/_solib_k8/libabsl_Sbase_Slibthrow_Udelegate.so: error: undefined reference to 'std::runtime_error::runtime_error(char const*)'
bazel-out/k8-fastbuild/bin/_solib_k8/libabsl_Sbase_Slibthrow_Udelegate.so: error: undefined reference to 'std::range_error::range_error(char const*)'
bazel-out/k8-fastbuild/bin/_solib_k8/libabsl_Sbase_Slibthrow_Udelegate.so: error: undefined reference to 'std::overflow_error::overflow_error(char const*)'
bazel-out/k8-fastbuild/bin/_solib_k8/libabsl_Sbase_Slibthrow_Udelegate.so: error: undefined reference to 'std::underflow_error::underflow_error(char const*)'
bazel-out/k8-fastbuild/bin/_solib_k8/libexternal_Scom_Ugoogle_Ugoogletest_Slibgtest.so: error: undefined reference to 'std::runtime_error::runtime_error(char const*)'
bazel-out/k8-fastbuild/bin/_solib_k8/libexternal_Scom_Ugoogle_Ugoogletest_Slibgtest.so: error: undefined reference to 'std::__throw_out_of_range_fmt(char const*, ...)'
bazel-out/k8-fastbuild/bin/absl/container/_objs/inlined_vector_test/inlined_vector_test.pic.o:inlined_vector_test.cc:function std::istream_iterator<int, char, std::char_traits<char>, long>::_M_read(): error: undefined reference to 'std::basic_ios<char, std::char_traits<char$ >::operator bool() const'
bazel-out/k8-fastbuild/bin/absl/container/_objs/inlined_vector_test/inlined_vector_test.pic.o:inlined_vector_test.cc:function std::istream_iterator<int, char, std::char_traits<char>, long>::_M_read(): error: undefined reference to 'std::basic_ios<char, std::char_traits<char$ >::operator bool() const'
collect2: error: ld returned 1 exit status
Target //absl/container:inlined_vector_test failed to build
Use --verbose_failures to see the command lines of failed build steps.
INFO: Elapsed time: 33.227s, Critical Path: 9.28s
INFO: 69 processes: 69 linux-sandbox.
 I was hoping to reproduce the same error frm the test, but it looks like building the absl test fails in itself. Maybe @adisuissa can write a simple inlinedvector test (no headermap stuff, just replicating the initialization and usage of variant and inlined vector in headermap) for you to add to an Envoy test and run in your environment?

Big thanks to @asraa and @adisuissa for helping out on this one. I was able to install GCC 9.1.1 in my Red Hat 7 environment and now I'm able to compile without errors using bazel build -c opt --copt=-DENVOY_IGNORE_GLIBCXX_USE_CXX11_ABI_ERROR=1 //source/exe:envoy-static.stripped and the resultant binary runs on RHEL7.

FWIW: I am still getting this error in debian-bullseye (gcc 9.x), I have to use the following options:
--copt="-Wno-redundant-move"
and in debian-buster (gcc 8.x):
--copt="-Wno-maybe-uninitialized" --copt="-Wno-uninitialized"


publish vscode extension as a public preview

follow up of #2324

[ ] setup CI/CD to package kiota with the extension and upload it.

Harcoded path portability https://github.com/microsoft/kiota/pull/2324#discussion_r1113918672

[ ] lint the typescript code

Possibly investigate string enums https://github.com/microsoft/kiota/pull/2324#discussion_r1113917903

[ ] fill in missing publishing information (publisher, description, etc)
[ ] record a screencast gif so people know what to expect.
[ ] augment npm tasks to start dotnet watch output to media so debug is easier

In terms of packaging we have the following options:

use the vs code dotnet SDK extension as an extensionDepencies (package.json). This would be great as we could only dotnet tool install after that and we'd not need to bundle anything with the extension, making it super lightweight. But it's an early preview that doesn't look to be maintained for now.
Build a script to publish kiota variants in the media folder. This would work but would results in a 150MB extension because of all the variants including the dotnet framework.
Download kiota on init like omnisharp does a bit more complex, but should get things done.

First public preview would be a release artifact in Github
Second preview will be a fully published vscode in the marketplace but is out of scope for now.


Module versioning support

Currently it's not possible to specify module version that should be loaded upon knot.x start.
Let's say we have the following setup:

knot.x config

{
    "config": {
        "...": "..."
    },
    "modules": [
        "knotx:io.knotx.KnotxServer",
        "knotx:io.knotx.HttpRepositoryConnector",
        "knotx:io.knotx.FragmentSplitter",
        "knotx:io.knotx.FragmentAssembler",
        "knotx:io.knotx.ServiceKnot",
        "knotx:io.knotx.ActionKnot",
        "knotx:io.knotx.HandlebarsKnot",
        "knotx:io.knotx.HttpServiceAdapter",
        "knotx:io.knotx.ResponseProviderKnot",
        "knotx:io.knotx.GatewayKnot",
        "knotx:example.search.images.index"
    ]
}

directory structure

.
├── app
│   ├── knotx.jar -> /opt/knotx/author/knotx-standalone-1.1.0.fat.jar
│   ├── search-images-index-0.1.0-SNAPSHOT-knotx.jar -> ../extensions/search-images-index-0.1.0-SNAPSHOT-knotx.jar
│   ├── search-images-index-0.2.0-SNAPSHOT-knotx.jar -> ../extensions/search-images-index-0.2.0-SNAPSHOT-knotx.jar
│   ├── search-images-index-0.2.1-SNAPSHOT-knotx.jar -> ../extensions/search-images-index-0.2.1-SNAPSHOT-knotx.jar
│   └── search-images-index-0.3.0-knotx.jar -> ../extensions/search-images-index-0.3.0-knotx.jar
├── config
│   ├── author.json
│   └── publish.json
├── extensions
│   ├── search-images-index-0.1.0-SNAPSHOT-knotx.jar
│   ├── search-images-index-0.2.0-SNAPSHOT-knotx.jar
│   ├── search-images-index-0.2.1-SNAPSHOT-knotx.jar
│   └── search-images-index-0.3.0-knotx.jar
├── knotx.conf
├── knotx-standalone-1.1.0.fat.jar
└── logback.xml

app/* is defined as classpath for that knot.x instance.
As listed above we have 4 different versions of the same module and just one reference to knotx:example.search.images.index in the config file (w/o version). Because of that it's not possible to determine which version of given module will be used.
I can see a couple of scenarios here:

the first (hypothetically the oldest) loaded module will be used (with an assumption that modules are loaded/added to classpath in alphabetical order)
the last (hypothetically the newest) loaded module will be used (same assumption as above)
module that was loaded first (I'm not sure if filename matters while loading modules from classpath)
random one will be used (may change with each knot.x restart)
knot.x fails to start as it's not possible to determine which module version should be loaded

Workaround for now is to use the same filename for each module, so there's no chance for having more than 1 version of given module. That doesn't sound like a long term solution though.

@jwadolowski looks like it would be a nice feature. However, it's not possible to make it happen because it's not a OSGi. The reason behind that is the mentioned app/ folder is simply a folder from where all classes is to be loaded by regular JVM classloader - it's not a module folder. The same issue would be if you put multiple versions of jar classes into tomcat - you would endup with classloader issue that unknown version of the same jar would be available for your code.
The good practice is to have just one version of jar in app folder


Show Delete Warning Modal for Delete Action

This might be as simple as using openDeleteAllItemsModal for the "Delete" action's method.

Commit: https://github.com/johneastman/todo/commit/6969ffec80ede6a0a1b4b88204555b8e01350d6a


Delegate Expressions - Context querying (v2)

In v2 you made DbContext available.
modelBuilder.Filter("Notes_CurrentUser", (Note n) => n.PersonID, (MyContext ctx) => ctx.CurrentPersonID);
I have complex permission logic and one-to-many child properties are not available. Is the code below unsupported or it is but I'm doing something wrong?
modelBuilder.Filter("Filter",
    (User u, List<int> allowed) => allowed.Contains(u.UserId),
    (RegLinkEntities db) => db.UserAccess
        .Where(i => i.UserId == Context.Current.UserId)
        .Select(i => i.AccessedUserId)
        .ToList());

Assuming "RegLinkEntities" is a DbContext derived class, it appears that you are executing a db query from inside the filter.  I don't think I would recommend that, but I think it would work.
If it's not, then I would need more information about the problem.

I know it's not good. Now when .Any() is supported I don't need to try this hack. Issue was on doing Async query with querying context in Func<TConext, T>. It was throwing "A second operation started on this context before a previous asynchronous operation completed."
What would be really helpful instead, is to query context on the filter expression itself to manage connections without foreign keys.
modelBuilder.Filter("Value_NoForeignKey",
    (Note n, MyContext context) => context.Users.Any(u => u.ID == n.PersonID)
);

The filter expressions are evaluated when the query parameters are being set.  So executing a query from inside the filter will not work.
Is there a reason you are avoiding making a foreign key?  What you are trying to do there is to enforce the FK relationship on every query operation.  It would be much more efficient to only do that during write operations using a FK since (I assume) you would be doing far less write operations than read.

Laziness? :) I already realized it is not going to work like that and I converted all calculated values to real values within the record.


One of my code while running it is not running

Thanks for creating this issue! We figured it's missing some basic information or in some other way doesn't follow our issue reporting guidelines. Please take the time to review these and update the issue.
Happy Coding!

This issue has been closed automatically because it needs more information and has not had recent activity. See also our issue reporting guidelines.
Happy Coding!


modify service overview to use right module and connection paramaters

@rloutlaw,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs-ref-services/mysql.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Create league / hill system

Massive story to create a hill system.
Ideas include:

Ranked placement of users
Beginner hills
Restricted instruction set hills
Hill visualisation / view of results
Stats / badges / bragging rights
Social network integration?

Idea which can be added:
Continously/never ending/endlessly running "evolver hill".
Evolver has to be written in redcode spec2009 (Skybuck spec). All evolvers are run before the start of each battle.
The evolver may run a certain number of redcode cycles to generate a warrior.
The evolver's starting point could be location 0.
The warrior to be generated would be located at coresize-100.
Next the software copies the warriors from that location for battling.
Evolvers are then rated/score on how their warriors perform ! ;)
(Advanced features could include, some kind of access to the core memory while warriors run for observation purposes, statistics gathering, other idea could be to copy another warrior into evolver code located at coresize-200 so that evolver can "breed" with other generated warriors")
(Last radical idea is to allow "evolvers" to breed with each other, but this idea has not been explored further and just came to me somewhat... but perhaps this is going too far but could still be tried if found interesting, Evolvers could then be limited to size 4000 or so, but both codes placed in 8000 core and can access each other, or core could be doubled in size to 16.000 but that may upset warrior code, so maybe keeping it the same size as actually battle core would be best).

Other ideas could be blockchain hill to reward hill computations with coins, and also winners with coins.


[issue] Broken dependency smtpd

Describe the bug
Project depends on github.com/prologic/smtpd, but https://github.com/prologic/smtpd results in a 404, repo seems gone.
Error occurred when trying to open the Nuclei source in GoLand on a pristine machine.
Version
Nuclei: Trunk of master (currently at 84244b5)
Interactsh: Seems to be present in trunk, too
Screenshot of the error or bug
go list -modfile=nuclei/v2/go.mod -m -json -mod=mod all #gosetup
go<EMAIL_ADDRESS>requires
<EMAIL_ADDRESS>invalid version: git fetch -f origin refs/heads/*:refs/heads/* refs/tags/*:refs/tags/* in ~/go/pkg/mod/cache/vcs/65b3100cfa8e2061b6047e41aaceb4a1e850f70977a718cfde5bd0e009bb0722: exit status 128

I'm not a Go person, so I'm not used to some probably Go-specific routines. I had a look at https://pkg.go.dev/git.mills.io/prologic/smtpd - they refer to https://git.mills.io/prologic/smtpd/ as their repository... and indeed, the latest commit (2 weeks ago) states that the project has moved away from GitHub to Mill. So it's just a matter of updating the reference, I guess :-)

@Skyr Thanks for reporting the issue! This will be fixed asap.

@Skyr Thanks for reporting this; the fix has since been merged into the dev branch; for the time being, you can alter the nuclei mod file to point to the interactsh dev branch.

@Skyr we pushed a new release for - https://github.com/projectdiscovery/interactsh/releases/tag/v0.0.4, thanks for reporting.


Could not find or load main class coursier.bootstrap.launcher.Launcher

Hello,
I'm trying to setup a test jupyterhub with nbgrader instance that uses a scala kernel.
I've been following the instructions and I can't seem to get it to work.
I'm installing almond with the --global option as if it's ommited jupyter kernelspec list does not find the scala kernel.
The installation steps I followed (after downloading the courier file using curl as per the instructions)
jupyter kernelspec list
Available kernels:
  python3    /usr/local/share/jupyter/kernels/python3
gordon@jupyter-playground:~$ sudo ./coursier launch --fork almond -- --install --global
[sudo] password for gordon:
Installed scala kernel under /usr/local/share/jupyter/kernels/scala
gordon@jupyter-playground:~$ jupyter kernelspec list
Available kernels:
  python3    /usr/local/share/jupyter/kernels/python3
  scala      /usr/local/share/jupyter/kernels/scala

Upon launching a jupyter notebook instance, I do indeed see the scala kernel

But after a couple of seconds, I it gets disconnected, the Scala icon disappears and the text goes to 'No Kernel'
Jupyter notebook console I get the error :
[I 2023-08-11 13:00:14.872 ServerApp] 200 GET /user/instructor1/api/sessions?1691751614855 (instructor1@::ffff:<IP_ADDRESS>) 2.65ms
[I 2023-08-11 13:00:14.888 ServerApp] 200 GET /user/instructor1/api/kernelspecs?1691751614877 (instructor1@::ffff:<IP_ADDRESS>) 3.40ms
Error: Could not find or load main class coursier.bootstrap.launcher.Launcher
Caused by: java.lang.ClassNotFoundException: coursier.bootstrap.launcher.Launcher

Followed by multiple retries until a "Kernel Does not Exist" error is thrown.
I've tried with almond kernel 0.10.0 but had a similar error, instead it was a 'Class almond.ScalaKernel not found' error.
My java version output is :
java -version
openjdk version "11.0.20" 2023-07-18
OpenJDK Runtime Environment (build 11.0.20+8-post-Ubuntu-1ubuntu122.04)
OpenJDK 64-Bit Server VM (build 11.0.20+8-post-Ubuntu-1ubuntu122.04, mixed mode, sharing)

I'm at a loss as to what to do here, any help or ideas would be appreciated.

Hello,
I am having the same issue here.

This is mentioned somewhere but the short version is: Gotta make sure the notebook can actually find the coursier cache.
I used
ENV COURSIER_CACHE=/usr/share/coursier/cache

in a dockerfile to set this.
See related issue: https://github.com/almond-sh/almond/issues/729


metric privacy filters

Implement privacy filters for features that do not hit the minimum thresholds defined in the Micromobility Metrics Specification.

This shipped.


Store PBKDF2

Hi,
Using this dart's package, I would like to know how to store the PBKDF2 algorithm because every time I close the app and I have to get back the hash, It gives me another algorithm.
Thanks

Hi,
Every time you call Password.hash it generates a different hash because of salting, that is part of security. But whenever you call Password.verify if this generated hash matches the given plain text, it should properly work.
That is one-way cryptography.
If you are looking for two-way cryptography, like generating always the same encrypted message from the given inputs so you can decrypt this message using the same inputs (key+iv), please, checkout: https://github.com/leocavalcante/encrypt

Great! I like this encrypting way, I use Salsa20, and when I store encrypted.base64, after read this back, Can I get "encrypted" from "encrypted.base64"?

I just saw the method Encrypted.from64(string encrypted), thanks for all ;)


Make Genesis keypair well-know, remove getGenesisKeyPair()

It is quite obvious in hindsight that the genesis keypair was the first well-known key,
and should be accessible through the same mean.
This will also allow unified lookup for the testing utilities being developed.

source/agora/utils/Test.d(821,41): Error: undefined identifier UnitTestGenesisAddressUbyte

Just this needs to be fixed.

Should be fixed

Third time's a charm


wip: sub-commands

not yet finished, but feel free to give me some feedback whether you like the idea of having sub commands or not?
usage: beancounter [<flags>] <command> [<args> ...]

A command-line Bitcoin wallet balance audit tool.

Flags:
  --help   Show context-sensitive help (also try --help-long and --help-man).
  --debug  Enable debug output.

Commands:
  help [<command>...]
    Show help.

  keytree [<flags>] <i>...
    Performs one or more child key derivations.

  find-addr [<flags>]
    Finds the change/index values for a given address.

  find-block [<flags>] <timestamp>
    Finds the block height for a given date/time.

  compute-balance --type=TYPE [<flags>]
    Computes balance for a given watch wallet.

Pull Request Test Coverage Report for Build 144

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 39.841%

Totals

Change from base Build 123:
0.0%

Covered Lines:
400

Relevant Lines:
1004

💛  - Coveralls

Pull Request Test Coverage Report for Build 144

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 39.841%

Totals

Change from base Build 123:
0.0%

Covered Lines:
400

Relevant Lines:
1004

💛  - Coveralls

I was thinking about sub-commands for a while now, and I think they would help a lot with simplifying the flags as some of them only apply to certain actions (e.g. beancounter keytree is a useful subcommand that doesn't need a backend). 👍


Open file from COMMIT view

I think it would be useful for chain like this:
show commit > open file > show log of this file
Open file works from DIFF view by 'o' shortkey but doesn't work from COMMIT view.

Just linked this issue to #102.  This feature is a subset of the functionality described there, so closing this ticket in favor of the other.


Enable support for both JDK1.8 and JDK1.7

Current plugin is built using JDK1.8 and its not backward compatible on JDK1.7. Enable support for both JDK1.8 and JDK1.7.

Classifier approach did not quite work for non project based plugin so downgrading the JDK to 1.7


DROOLS-1174 reproducer

This commit adds missing test coverage for the fix made some time ago.
I believe this is a valuable addition as the test fails when the fix is reverted and there was no such failing test before the bug was fixed and no test was included with the fix.

Note: the test method name should be changed before merging.

@yurloc Good job. Please rename the test method testReorderRightMemoryOnIndexedField and I'll merge it.

Jenkins, retest this. Let's give it one more try but I guess it's just unstable tests because all tests pass on my machine.

@mariofusco method name updated, please go ahead and merge.


application failure if base URL does not begin with valid protocol (http:// etc)

If the protocol name is left out of the base URL field under settings, the gitbucket application fails with
java.net.MalformedURLException: unknown protocol: example.com

Do you think how should GitBucket work in that situation?


[New game]: Memory Game

🎮 Game Request
In this game the user will have to flip the tiles and find all the matching pair of tiles inorder to win the game.
Point down the features
It is a game made using HTML,CSS and JavaScript.
Select program in which you are contributing
hacktoberfest
Code of Conduct

[X] I follow CONTRIBUTING GUIDELINE of this project.

Hey @AnkitaSikdar005 !
Thank you for raising an issue 💗
You can self assign the issue by commenting /assign in comment 😀
Make sure you follow CODE OF CONDUCT and CONTRIBUTING GUIDELINES 🚀
Don’t Forget to ⭐ our GameZone🎮
Make sure you join our Discord🕹

/assign

/assign

Hey @rituraj12797 ! , This issue is already assigned to @AnkitaSikdar005. 💗
You can work on other issues 🚀
If there is anything wrong you can contact us on Discord🕹👀

Hello @AnkitaSikdar005, Time's up!⏰
Sorry for closing your issue!
But it's more than a week since we haven't received anything from your side 😢 .
Come up with new ideas, create a new issue and make sure you finish it within a week! 🔥
All the best! 🚀
Happy Hacking! 💗

Hey @AnkitaSikdar005 ! Thank you so much for your raising the issue💗
It’s all yours, you can come anytime again and make some contributions! 🚀
Alone, we can do little, but together we can do so much! 😇


test: remove old fixme from the tests

#16640 is closed and the annotation isn't necessary anymore.

Thanks!

Sorry for the delay....


Choose the Certificate tab, and then select Import. !!! NO IMPORT OPTION

"Step  3 : Choose the Certificate tab, and then select Import. "
There is no "Import" option in   SQL Configuration Manager

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: be811156-078f-e4d9-406a-7e63e06603b5
Version Independent ID: e7ab123c-1f76-14ac-2d55-296e7845a50c
Content: Certificate Management (SQL Server Configuration Manager) - SQL Server
Content Source: docs/database-engine/configure-windows/manage-certificates.md
Product: sql
Technology: configuration
GitHub Login: @markingmyname
Microsoft Alias: maghan

ONLY in 2019 version


⚠ Samsung has degraded performance

In e91150a, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 2397 ms

Resolved: Samsung performance has improved in 4adff7f.


Unable to install dependencies

Great project, I will be saving a lot of time using it.
kirimase add had issues while trying to install dependencies. When I use shadcn-ui cli package to install deps it works fine to call pnpm.
Error:
$ pnpm kirimase add
? Select a package to add
Auth.js (NextAuth)
? Select a provider to add
github
[15:22:42] ✔ File created at src/app/api/auth/[...nextauth]/route.ts
[15:22:42] ✔ File created at src/lib/auth/Provider.tsx
[15:22:42] ✔ File created at src/lib/auth/utils.ts
[15:22:42] ✔ File created at src/lib/db/schema/auth.ts
[15:22:42] ✔ File created at src/components/auth/SignIn.tsx
[15:22:42] ◐ Installing packages: @auth/core @auth/drizzle-adapter next-auth ...
node:events:491
      throw er; // Unhandled 'error' event
      ^

Error: spawn pnpm ENOENT
    at ChildProcess._handle.onexit (node:internal/child_process:283:19)
    at onErrorNT (node:internal/child_process:476:16)
    at process.processTicksAndRejections (node:internal/process/task_queues:82:21)
Emitted 'error' event on ChildProcess instance at:
    at ChildProcess._handle.onexit (node:internal/child_process:289:12)
    at onErrorNT (node:internal/child_process:476:16)
    at process.processTicksAndRejections (node:internal/process/task_queues:82:21) {
  errno: -4058,
  code: 'ENOENT',
  syscall: 'spawn pnpm',
  path: 'pnpm',
  spawnargs: [ 'install', '@auth/core', '@auth/drizzle-adapter', 'next-auth' ]
}

Node.js v18.16.0

Hey! Thanks for this. First open-source package for me so still learning the ropes! I've changed the logic from spawn to execa (a better version of spawn). I'm hoping that's solved it. Let me know if that's resolved the issue.

No prob, keep up the good work.
Can confirm that execa helped and that the package is working as expected!


Add usage with async/await

Added a usage example for async/await usage and I also changed the var to const due to const truly being constant in modern JS while var is re-assignable and you don't want to reassign a module import.
More info on async/await and JS concurrency (youtube)
Concerning let, var and const in the ES6 spec (medium)
Semantic versioning classification:

[ ] This PR changes the library's interface (methods or parameters added)

[ ] This PR includes breaking changes (methods removed or renamed, parameters moved or removed)

[X] This PR only includes non-code changes, like changes to documentation, README, etc.

Coverage remained the same at 76.923% when pulling 1e8db757e5a47d84aad3713277c7f4257659d97b on Favna:patch-1 into b2fa6fa050545eec18160fd40a3c316d89660494 on jimf:master.

Thanks! 🍻


Spell Check No Longer Working

Hello, miss-spelled words are no longer underlined on version 2.0.0 - running Chrome 75.0.3770.142 on MacOS. Will this be permanently fixed at some point?

Hey @vrl2 , sorry I have taken so long to respond. This should be fixed as of v3.0.0 which I just released. Please download that or update your current app and give it a try. If it still doesn't work, please feel free to open a new issue.


fix: version info of released binaries

closes #222

Codecov Report

Merging #243 (f611735) into develop (dff0f00) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           develop     #243   +/-   ##
========================================
  Coverage    34.42%   34.42%
========================================
  Files           49       49
  Lines         4424     4424
========================================
  Hits          1523     1523
  Misses        2750     2750
  Partials       151      151

Flag
Coverage Δ

unittests
34.42% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update dff0f00...f611735. Read the comment docs.


Add ability to set ShortName at Type

In https://github.com/Gabriel439/Haskell-Optparse-Generic-Library/pull/36, the ability to set a short name was added via Modifiers.  In https://github.com/Gabriel439/Haskell-Optparse-Generic-Library/issues/41, there is a request to make sure these short names are unique.  This PR attempts to enhance short-name creation with a type decorator; while it does not prevent duplicate short names, it makes dealing with them much easier.
This PR adds a new decorator <$> ("$" signaling "S", which stands for "Short name", although it can certainly be changed if the connection to fmap is too strong) that allows the short name of a field to be specified in its type.  This short name supersedes any that is set by Modifiers; this way, one can use a standard short-name modifier (say, firstLetter), and then manually replace short names of specific fields in the type.  Adding <$> "" to a type removes the short name altogether.
Unfortunately, there are no type-level Chars, which means that we're forced to use type-level strings (i.e., Symbol); in this implementation, I opt to ignore all but the first character.

Yeah, I'm worried about <$> conflicting with the one exported from the Prelude.  Would it be possible to rename this operator to <#>?

Okay, I changed <$> to <#>.  How does that look?

Looks great!  Thank you for contributing this 🙂


performing sync through geogig remotes fail for https protocol

Is this fixed in https://github.com/ROGUE-JCTD/MapLoom/commit/734cde82b9601683215ca9a7bf5488f3b3b0c9d0?

Fixed in maploom but should re-test when geoshape 1.5 is updated to use latest geogig containing end transaction fix.


Release Testing: non-utils in /utils

/utils/resilience-demo
and
/utils/dos-dumb-object-server
are not user-relevant at the moment. Perhaps they ought to be moved out of /utils and into somewhere else like /demos and /testing/tools?
cc @slfritchie
cc #2422

Uff, the commit that renamed utils/resilience-demo to demos/resilience-demo got lost, sorry.

Hrm, no, the directory renaming commit didn't get lost, it's in the release-0.5.3 branch at 76ce6177afe7e6deec4da7571273d16406d8b431.
I don't have a strong opinion about where the dos-server.py and journal-dump.py scripts ought to go.

Ah, my bad. I must have been looking at the wrong branch when I checked this.
Are we supporting dos-server in any way? Do we have docs or release notes on it? If not, I think it should probably move to testing/tools.


[Feature Request] Splash Screen for Portable executable

My understanding is that the portable executable extracts the application to a temp folder to run it.
The problem I have is that the users are stuck waiting for the extraction process before they see the application has launched. The process is rather fast on my flash PC with its SSD and ridiculous CPU clock speeds but when you start running on a PC that is still running a 3rd Gen i3 CPU and HDD that's equally as old, you find that people are impatient and often end up opening the executable 4-5 times before the initial one opens.
Therefore, my proposal is that there be an option to include a splash screen/image that shows when the portable application executable is opened to indicate that something is actually happening in the background.

Thanks for filing an issue, but this is outside of the scope of Electron Packager. If you'd like a splash screen during installation, you'll want to configure this sort of thing in, for example on Windows, electron-winstaller.

@malept I think you missed the point here. His use case, is for a portable application (not an installer). The process of unziping the app may take several seconds depending on the app size. We would like for the user to see a splash screen as soon as the app executable is double clicked.


Issue with Xcode 9

Hi there, it appears this particular dependancy is unsupported on Xcode 9. It would be great if this could be addressed as I would imagine it's causing issues on a lot of plugins that rely on your great work. Thanks :)

I'm using XCode 9.0.1 but don't get any issues with this library


Exceptions

Thanks for taking the time to update things for Live Preview!
I've found some exceptions where it will not shorten the link in Live Preview:

When the link is in a bullet point:

- [[File#Heading]]

When the text has formatting applied (no matter what type):

*Sentence [[File#Heading]] with formatting*

When the link is in a block quote:

> [[File#Heading]]

I think this issue should be now fixed in 1.0.1! Please let me know if you encounter any more edge cases.
The only other edge case that I am aware of that isn't fixed at the moment is when links are used in mermaid diagrams.


New version: ComplexityMeasures v2.9.1

Registering package: ComplexityMeasures
Repository: https://github.com/JuliaDynamics/ComplexityMeasures.jl
Created by: @Datseris
Version: v2.9.1
Commit: 60ad04b94a4a01fd69dbe0670c912182d788bd0d
Git reference: HEAD
Description: Estimators for probabilities, entropies, and other complexity measures derived from observations in the context of nonlinear dynamics and complex systems

Urg, please close this!


SSM access to Fargate ECS

Add ability to have SSM access to Fargate (ECS)

Test:

 Add ability to have SSM access to Fargate / ECS sidecar containers (like datadog but for SSM


Fix typos.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

License check is ok.


Updating Hungarian intentions

The current intentions are unusable, as they do not employ conjugations at any level. The entire language, which utilizes conjugations, has been updated.
Due to the nature of the Hungarian language, there's an issue where word roots change with conjugation. (For example, if the room is a kitchen, instead of "kapcsold be az összes ventilátort a konyhaban", it should be "konyhában".)
I circumvented these with suffix words, like using "piros színre" (to red colour) instead of "pirosra" (to red), or by changing the word order to avoid these issues.
But for correct and complete Hungarian expressions, it's worth adopting assistant aliases for those room names whose endings change during conjugation (thankfully, not all are like this), such as "szoba" (room), "konyha" (kitchen). For these, "szobá" and "konyhá" aliases should be added, and this way, even the sentence "kapcsold be az összes ventilátort a konyhában" will be usable (this is much more Hungarian-like).
The tests have successfully run, I hope there won't be any problems :)
Good luck with its use.
Viktor
python3 -m script.intentfest validate --language hu
Language: hu
[WARN] tests/hu/homeassistant_HassNevermind.yaml: 1 test(s) missing response check
[WARN] responses/hu/HassGetState.yaml: unused response default

All good!

Please run script/lint locally on your branch to learn what needs to be changed, then add a commit that will make it pass the linting.

@tetele Hello,
I ran it and got the error below.
Does this now mean there is an error in the files below? or did the files below succeed and I should look for which one the check didn't succeed?
Sorry, this is my first time doing this :)
[INFO] Initializing environment for https://github.com/pre-commit/mirrors-prettier.
[INFO] Initializing environment for<EMAIL_ADDRESS>[INFO] Installing environment for https://github.com/pre-commit/mirrors-prettier.
[INFO] Once installed this environment will be reused.
[INFO] This may take a few minutes...
prettier.................................................................Failed
- hook id: prettier
- files were modified by this hook

responses/gl/HassTurnOff.yaml
sentences/pl/script_HassTurnOn.yaml
tests/es/person_HassGetState.yaml
sentences/ka/cover_HassTurnOff.yaml
[...]
and many more...

Your only concern are thr files in hu dirs. The linter should automatically edit the files with issues (spacing issues, long lines etc.). After running the linter, they should already be updated and ready to be committed. Once you do, your CI pipeline should run without errors if you've properly written tests.


I18n of the theme

Would it be possible to allow the I18n?
For example, react-i18next seems like a good fit for this purpose.
Our users speaks mainly French and we have no simple way to translate the elements wrote in English.

Hmm this would be quite difficult as much of the copy comes from swagger-ui. If support for l18n comes to swagger-ui, it would then be possible to add it to this theme.
see: https://github.com/swagger-api/swagger-ui/issues/2488


al.optimizely.get is not a function

We were comparing our Snowplow pageviews against Google Analytics pageviews and wondering why Snowplow was always lower after we started to use 2.8.2 and Optimizely Summary flags in JS. Is anyone else getting this? This optimizely error inside Snowplow Tracker is causing pageview event not to fire, which probably explaining why we are lower than GA. I’m using chrome.
sp.js:63 Uncaught TypeError: al.optimizely.get is not a function
    at aI (sp.js:63)
    at aa (sp.js:63)
    at a2 (sp.js:66)
    at bp (sp.js:61)
    at aK (sp.js:67)
    at sp.js:73
    at ae (sp.js:69)
    at Object.trackPageView (sp.js:73)
    at s (sp.js:29)
    at new d.InQueueManager (sp.js:29)

from one of my lead developers:
because that seems to be the problem… the getOptimizelyXData function in the SP tracker looks for optimizely to be defined, and then tries to call optimizely.get, which has been added in the optimizelyX JS API
(the getOptimizelyData function for the normal optimizely summary looks for both optimizely and optimizely.data to be defined before attempting access)
and that’s getting called in the initialization of the tracker… so if it errors out there, we get nothing.

Hey @markjensen74, can you provide some more context? Are you using both optimizelyXSummary and optimizelySummary?

@chuwy yes we have both enabled at the same time.  here is our config:
}(window, document, 'script', '//d1fc8wv8zag5ca.cloudfront.net/2.8.2/sp.js', 'GAplow'));
window.GAplow('newTracker', 'cf-hosted-v2.8.2', this.spCollector, {
  appId: this.appId,
  platform: 'web',
  cookieDomain: '.generalassemb.ly',
  discoverRootDomain: true,
  cookieName: 'sp',
  encodeBase64: false,
  respectDoNotTrack: true,
  userFingerprint: true,
  userFingerprintSeed:<PHONE_NUMBER>,
  pageUnloadTimer: 0,
  forceSecureTracker: false,
  useCookies: true,
  post: true,
  bufferSize: 1,
  maxPostBytes: 45000,
  cookieLifetime: 86400 * 180,
  contexts: {
    webPage: true,
    gaCookies: true,
    optimizelyXSummary: true,
    optimizelySummary: true
  }
})

window.GAplow('enableActivityTracking', 30, 30);
window.GAplow('trackPageView');
window.GAplow('enableErrorTracking');
if (this.userCookieName) window.GAplow('setUserIdFromCookie', this.userCookieName);

}

Thanks, we're looking into it.

We'll get this scheduled! Thanks for raising

any updates on this?


Preformance questions

Hello, I was wondering how accurate this is and how long it takes for the program to analyze a photo. How many faces can it handle?

Hey ! Personally I've never tried having more than 3 faces in the frame.
On Fri, Jan 26, 2018 at 11:47 AM, Ben<EMAIL_ADDRESS>wrote:

Hello, I was wondering how accurate this is and how long it takes for the
program to analyze a photo. How many faces can it handle?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/vyomshm/face_recognition/issues/1, or mute the thread
https://github.com/notifications/unsubscribe-auth/ATP_cjWKT3SXpMWz2UOcm17q8WpAuvdJks5tOW4AgaJpZM4Rt5mL
.


⚠ Samsung has degraded performance

In c3dd535, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 2280 ms

Resolved: Samsung performance has improved in 9e364d1.